From 07ad69cebfd72c8d9a9033ad963fdecc6b4b4981 Mon Sep 17 00:00:00 2001
From: Edison Su <sudison@gmail.com>
Date: Wed, 6 Nov 2013 15:32:19 -0800
Subject: [PATCH] CLOUDSTACK-5737: isExternalSnapshot should return true for
 CLVM and RBD only

---
 .../cloud/hypervisor/kvm/storage/LibvirtStoragePool.java   | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

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 df0af5f0530..32aa6030167 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