----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8534/#review14501 -----------------------------------------------------------
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java <https://reviews.apache.org/r/8534/#comment30877> This function should be put into storagepoolmgr, you can add a new function, getsecondarystoragefromuri(string uri) - edison su On Dec. 11, 2012, 10:31 p.m., Marcus Sorensen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/8534/ > ----------------------------------------------------------- > > (Updated Dec. 11, 2012, 10:31 p.m.) > > > Review request for cloudstack and edison su. > > > Description > ------- > > This is a refactor of a few things to allow pluggable storage adaptors for > KVM. Instead of using LibvirtStorageAdaptor for everything, you can create > your own storage adaptor and use it. We select storage adaptor based on > storage pool type, thus we needed to adjust LibvirtComputingResource to pass > pool type to everything in KVMStoragePoolManager. This in turn required that > we pass the info necessary to LibvirtComputingResource as well, so a few > agent Commands were modified. > > Note this patch in and of itself shouldn't change any existing behavior, just > allow for new storage adaptors to be selected based on storage pool type. > > > Diffs > ----- > > api/src/com/cloud/agent/api/BackupSnapshotCommand.java 9476d7d > api/src/com/cloud/agent/api/CreatePrivateTemplateFromSnapshotCommand.java > bf6f260 > api/src/com/cloud/agent/api/CreatePrivateTemplateFromVolumeCommand.java > b589ed5 > api/src/com/cloud/agent/api/CreateVolumeFromSnapshotCommand.java f006329 > api/src/com/cloud/agent/api/SnapshotCommand.java 892db37 > api/src/com/cloud/agent/api/UpgradeSnapshotCommand.java e451768 > api/src/com/cloud/agent/api/downloadSnapshotFromSwiftCommand.java 0586f8c > api/src/com/cloud/agent/api/storage/PrimaryStorageDownloadCommand.java > b563177 > > plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java > e0187e6 > > plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/KVMStoragePoolManager.java > 2c0e0ac > server/src/com/cloud/storage/StorageManagerImpl.java e252633 > server/src/com/cloud/storage/snapshot/SnapshotManagerImpl.java 32e37e6 > server/src/com/cloud/template/TemplateManagerImpl.java 1e87de2 > server/src/com/cloud/vm/UserVmManagerImpl.java 02d630f > > Diff: https://reviews.apache.org/r/8534/diff/ > > > Testing > ------- > > tested the following on a KVM cloudstack install: > > add primary storage > delete primary storage > add vm to CLVM primary storage > add vm to NFS primary storage > create data disk > create snapshot > create template > attach datadisk to VM > detach datadisk from VM > delete datadisk > delete template > delete snapshot > stop vm > delete vm > register iso > attach iso > detach iso > > > Thanks, > > Marcus Sorensen > >
