You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by ed...@apache.org on 2014/01/03 21:44:03 UTC
git commit: updated refs/heads/4.3 to 07ad69c
Updated Branches:
refs/heads/4.3 3964a566f -> 07ad69ceb
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/07ad69ce
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/07ad69ce
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/07ad69ce
Branch: refs/heads/4.3
Commit: 07ad69cebfd72c8d9a9033ad963fdecc6b4b4981
Parents: 3964a56
Author: Edison Su <su...@gmail.com>
Authored: Wed Nov 6 15:32:19 2013 -0800
Committer: edison <su...@gmail.com>
Committed: Fri Jan 3 12:43:16 2014 -0800
----------------------------------------------------------------------
.../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/07ad69ce/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 df0af5f..32aa603 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
@@ -181,11 +181,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