> On Aug. 10, 2012, 8:36 p.m., edison su wrote: > > Ship It! > > Marcus Sorensen wrote: > please ship r3, looks like I beat you by only a minute, so I'm not sure > you saw it...
I added a comment on Review board, but seems I forgot to submit it... Could you please resend a new patch, which based on both 4.0 and master HEAD? I manually resolved the conflict when I applied your patch, but not sure that's enough. - edison ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6470/#review10152 ----------------------------------------------------------- On Aug. 10, 2012, 8:35 p.m., Marcus Sorensen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/6470/ > ----------------------------------------------------------- > > (Updated Aug. 10, 2012, 8:35 p.m.) > > > Review request for cloudstack. > > > Description > ------- > > Adds CLVM as an option for primary storage. It copies most of the RBD bits, > in most cases simply adding an 'else if' or similar. In the managesnapshot.sh > we add the ability to manipulate snapshots for the CLVM volumes, as well as a > few minor fixes like adjusting the usage for the -b option to reflect that it > accepts four arguments (note that it already accepted four args, it just > wasn't reflected in the usage). > > > Diffs > ----- > > api/src/com/cloud/storage/Storage.java d6bd04e > > plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java > 4a22e73 > > plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/KVMStoragePoolManager.java > 491f772 > > plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/LibvirtStorageAdaptor.java > 85c64ba > > plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/LibvirtStoragePool.java > bc428e1 > scripts/storage/qcow2/managesnapshot.sh a305ba8 > server/src/com/cloud/storage/StorageManagerImpl.java c7dda00 > ui/scripts/system.js 9d9936c > ui/scripts/zoneWizard.js 261668f > > Diff: https://reviews.apache.org/r/6470/diff/ > > > Testing > ------- > > Create system VMs on CLVM : passed > Create instances on CLVM : passed > Create data volumes on CLVM : passed > Delete CLVM data volumes : passed > New CLVM vol snapshot : passed > Create CLVM vol from snapshot : passed > > > Thanks, > > Marcus Sorensen > >
