Updated Branches:
  refs/heads/4.2 9ca72536c -> c5131af65

CLOUDSTACK-3758 [Vmware][ZWPS] Failed to create volume from snapshot

Signed-off-by: Sateesh Chodapuneedi <[email protected]>


Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/c5131af6
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/c5131af6
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/c5131af6

Branch: refs/heads/4.2
Commit: c5131af658ca70466a9ea4407c7048348ef7b931
Parents: 9ca7253
Author: Sateesh Chodapuneedi <[email protected]>
Authored: Wed Jul 24 03:48:28 2013 +0530
Committer: Sateesh Chodapuneedi <[email protected]>
Committed: Wed Jul 24 03:57:05 2013 +0530

----------------------------------------------------------------------
 .../cloud/storage/snapshot/SnapshotManagerImpl.java    | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/c5131af6/server/src/com/cloud/storage/snapshot/SnapshotManagerImpl.java
----------------------------------------------------------------------
diff --git a/server/src/com/cloud/storage/snapshot/SnapshotManagerImpl.java 
b/server/src/com/cloud/storage/snapshot/SnapshotManagerImpl.java
index 4ea275d..1300f02 100755
--- a/server/src/com/cloud/storage/snapshot/SnapshotManagerImpl.java
+++ b/server/src/com/cloud/storage/snapshot/SnapshotManagerImpl.java
@@ -44,6 +44,7 @@ import 
org.apache.cloudstack.engine.subsystem.api.storage.ZoneScope;
 import org.apache.cloudstack.storage.datastore.db.PrimaryDataStoreDao;
 import org.apache.cloudstack.storage.datastore.db.SnapshotDataStoreDao;
 import org.apache.cloudstack.storage.datastore.db.SnapshotDataStoreVO;
+import org.apache.cloudstack.storage.datastore.db.StoragePoolVO;
 import org.apache.log4j.Logger;
 import org.springframework.stereotype.Component;
 
@@ -78,9 +79,10 @@ import 
com.cloud.projects.Project.ListProjectResourcesCriteria;
 import com.cloud.resource.ResourceManager;
 import com.cloud.server.ResourceTag.TaggedResourceType;
 import com.cloud.storage.CreateSnapshotPayload;
+import com.cloud.storage.DataStoreRole;
+import com.cloud.storage.ScopeType;
 import com.cloud.storage.Snapshot;
 import com.cloud.storage.Snapshot.Type;
-import com.cloud.storage.DataStoreRole;
 import com.cloud.storage.SnapshotPolicyVO;
 import com.cloud.storage.SnapshotScheduleVO;
 import com.cloud.storage.SnapshotVO;
@@ -1145,7 +1147,14 @@ public class SnapshotManagerImpl extends ManagerBase 
implements SnapshotManager,
         }
         String snapshotName = vmDisplayName + "_" + volume.getName() + "_" + 
timeString;
 
-        HypervisorType hypervisorType = volume.getHypervisorType();
+        HypervisorType hypervisorType = HypervisorType.None;
+        StoragePoolVO storagePool = 
_storagePoolDao.findById(volume.getDataStore().getId());
+        if (storagePool.getScope() == ScopeType.ZONE) {
+            hypervisorType = storagePool.getHypervisor();
+        } else {
+            hypervisorType = volume.getHypervisorType();
+        }
+
         SnapshotVO snapshotVO = new SnapshotVO(volume.getDataCenterId(), 
volume.getAccountId(), volume.getDomainId(), volume.getId(), 
volume.getDiskOfferingId(), snapshotName,
                 (short) snapshotType.ordinal(), snapshotType.name(), 
volume.getSize(), hypervisorType);
 

Reply via email to