etan39
/
cloudstack
mirror of https://github.com/apache/cloudstack.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

bug 12039,12041: when taking snapshot for detached volume, keep the worker VM around until backup copy to SSVM is done 

Reviewed-by: Frank
This commit is contained in:
Kelven Yang 2011-11-16 18:01:52 -08:00
parent ee9eadef47
commit ad6d7ff9d2
1 changed files with 17 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

33
core/src/com/cloud/hypervisor/vmware/manager/VmwareStorageManagerImpl.java
View File

@ -162,7 +162,7 @@ public class VmwareStorageManagerImpl implements VmwareStorageManager {
String snapshotBackupUuid = null;
VmwareContext context = hostService.getServiceContext(cmd);
VirtualMachineMO vmMo;
VirtualMachineMO vmMo = null;
try {
VmwareHypervisorHost hyperHost = hostService.getHyperHost(context, cmd);
@ -205,30 +205,31 @@ public class VmwareStorageManagerImpl implements VmwareStorageManager {
throw new Exception("Failed to take snapshot " + cmd.getSnapshotName() + " on vm: " + cmd.getVmName());
}
}
snapshotBackupUuid = backupSnapshotToSecondaryStorage(vmMo, accountId,
volumeId, cmd.getVolumePath(), snapshotUuid,
secondaryStoragePoolURL, prevSnapshotUuid,
prevBackupUuid,
UUID.randomUUID().toString().replace("-", ""));
success = (snapshotBackupUuid != null);
if (success) {
details = "Successfully backedUp the snapshotUuid: " + snapshotUuid + " to secondary storage.";
}
} finally {
if(vmMo != null)
vmMo.removeAllSnapshots();
try {
if (workerVm != null) {
// detach volume and destroy worker vm
workerVm.moveAllVmDiskFiles(dsMo, "", false);
workerVm.detachAllDisks();
workerVm.destroy();
}
} catch (Throwable e) {
s_logger.warn("Failed to destroy worker VM: " + workerVMName);
}
}
snapshotBackupUuid = backupSnapshotToSecondaryStorage(vmMo, accountId,
volumeId, cmd.getVolumePath(), snapshotUuid,
secondaryStoragePoolURL, prevSnapshotUuid,
prevBackupUuid,
hostService.getWorkerName(context, cmd));
success = (snapshotBackupUuid != null);
if (success) {
details = "Successfully backedUp the snapshotUuid: " + snapshotUuid + " to secondary storage.";
vmMo.removeAllSnapshots();
}
}
} catch (Throwable e) {
if (e instanceof RemoteException) {