apache/cloudstack
1
0
Fork 0
mirror of https://github.com/apache/cloudstack.git synced 2025-10-26 08:42:29 +01:00
Code Issues Packages Projects Releases Wiki Activity

bug 12039, 12041: detach all disks before destroy worker VM to avoid accidentally delete the underlying disk along with the worker VM. Reviewed-by: Kelven

Browse Source
This commit is contained in:
Kelven Yang 2011-11-15 19:04:54 -08:00
parent 37908d2669
commit 67a2320001
1 changed files with 8 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -507,8 +507,10 @@ public class VmwareStorageManagerImpl implements VmwareStorageManager {
return new Ternary<String, Long, Long>(installPath + "/" + templateName + ".ova", size, size);
} finally {
if(clonedVm != null)
clonedVm.destroy();
if(clonedVm != null) {
clonedVm.detachAllDisks();
clonedVm.destroy();
}
vmMo.removeSnapshot(tmpSnapshotName, false);
}
@ -656,8 +658,10 @@ public class VmwareStorageManagerImpl implements VmwareStorageManager {
clonedVm.moveAllVmDiskFiles(primaryDsMo, "", false);
clonedVm.detachAllDisks();
} finally {
if(clonedVm != null)
clonedVm.destroy();
if(clonedVm != null) {
clonedVm.detachAllDisks();
clonedVm.destroy();
}
}
}