----------------------------------------------------------- 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. Changes ------- clicked wrong diff file. 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 (updated) ----- 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