From e051490cea9ad659e05853fd8b38728c06dbfd8f Mon Sep 17 00:00:00 2001 From: Kelven Yang Date: Mon, 3 Feb 2014 14:01:10 -0800 Subject: [PATCH] Make sure to let VM question answer monitor exit gracefully --- .../hypervisor/vmware/mo/VirtualMachineMO.java | 17 +++++++++++------ 1 file changed, 11 insertions(+), 6 deletions(-) diff --git a/vmware-base/src/com/cloud/hypervisor/vmware/mo/VirtualMachineMO.java b/vmware-base/src/com/cloud/hypervisor/vmware/mo/VirtualMachineMO.java index 663adbff9cd..263c656868e 100644 --- a/vmware-base/src/com/cloud/hypervisor/vmware/mo/VirtualMachineMO.java +++ b/vmware-base/src/com/cloud/hypervisor/vmware/mo/VirtualMachineMO.java @@ -248,12 +248,17 @@ public class VirtualMachineMO extends BaseMO { } }); - boolean result = _context.getVimClient().waitForTask(morTask); - if (result) { - _context.waitForTaskProgressDone(morTask); - return true; - } else { - s_logger.error("VMware powerOnVM_Task failed due to " + TaskMO.getTaskFailureInfo(_context, morTask)); + try { + boolean result = _context.getVimClient().waitForTask(morTask); + if (result) { + _context.waitForTaskProgressDone(morTask); + return true; + } else { + s_logger.error("VMware powerOnVM_Task failed due to " + TaskMO.getTaskFailureInfo(_context, morTask)); + } + } finally { + // make sure to let VM question monitor exit + flags[0] = true; } return false;