CLOUDSTACK-5737: isExternalSnapshot should return true for CLVM and RBD only
Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/b1bba4b3 Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/b1bba4b3 Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/b1bba4b3 Branch: refs/heads/master Commit: b1bba4b3fbd5ae5ac421c4e28744ffa880b491b5 Parents: 7416988 Author: Edison Su <[email protected]> Authored: Wed Nov 6 15:32:19 2013 -0800 Committer: Anthony Xu <[email protected]> Committed: Fri Mar 28 16:14:59 2014 -0700 ---------------------------------------------------------------------- .../com/cloud/hypervisor/kvm/storage/LibvirtStoragePool.java | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/cloudstack/blob/b1bba4b3/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/LibvirtStoragePool.java ---------------------------------------------------------------------- diff --git a/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/LibvirtStoragePool.java b/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/LibvirtStoragePool.java index 550bd01..54edffc 100644 --- a/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/LibvirtStoragePool.java +++ b/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/LibvirtStoragePool.java @@ -178,11 +178,10 @@ public class LibvirtStoragePool implements KVMStoragePool { @Override public boolean isExternalSnapshot() { - if (this.type == StoragePoolType.Filesystem) { - return false; + if (this.type == StoragePoolType.CLVM || type == StoragePoolType.RBD) { + return true; } - - return true; + return false; } @Override
