Return-Path: X-Original-To: apmail-cloudstack-commits-archive@www.apache.org Delivered-To: apmail-cloudstack-commits-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id C8E1EFB33 for ; Wed, 17 Apr 2013 23:56:49 +0000 (UTC) Received: (qmail 7658 invoked by uid 500); 17 Apr 2013 23:56:45 -0000 Delivered-To: apmail-cloudstack-commits-archive@cloudstack.apache.org Received: (qmail 7593 invoked by uid 500); 17 Apr 2013 23:56:44 -0000 Mailing-List: contact commits-help@cloudstack.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@cloudstack.apache.org Delivered-To: mailing list commits@cloudstack.apache.org Received: (qmail 6985 invoked by uid 99); 17 Apr 2013 23:56:44 -0000 Received: from tyr.zones.apache.org (HELO tyr.zones.apache.org) (140.211.11.114) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 17 Apr 2013 23:56:44 +0000 Received: by tyr.zones.apache.org (Postfix, from userid 65534) id 3F9C247196; Wed, 17 Apr 2013 23:56:44 +0000 (UTC) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: alena1108@apache.org To: commits@cloudstack.apache.org Date: Wed, 17 Apr 2013 23:57:09 -0000 Message-Id: In-Reply-To: <7ef7a96fa4d34eebb0e5040c48976ee9@git.apache.org> References: <7ef7a96fa4d34eebb0e5040c48976ee9@git.apache.org> X-Mailer: ASF-Git Admin Mailer Subject: [27/32] git commit: updated refs/heads/internallb to 0cfe96b CLOUDSTACK-2071 - VirtualMachineManagerImpl.java start() method for instance can fail to start a VM without notifying caller, if no exception is triggered. The result is that VM start looks successful but was not. This fixes it by throwing an exception at the very end if the object to be passed back is still null. Signed-off-by: Marcus Sorensen 1366225829 -0600 Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/8d0bea99 Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/8d0bea99 Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/8d0bea99 Branch: refs/heads/internallb Commit: 8d0bea994dcd74cd6707b58ac2e4060a93f52d96 Parents: 9584815 Author: Marcus Sorensen Authored: Wed Apr 17 13:10:29 2013 -0600 Committer: Marcus Sorensen Committed: Wed Apr 17 13:13:05 2013 -0600 ---------------------------------------------------------------------- .../com/cloud/vm/VirtualMachineManagerImpl.java | 5 +++++ 1 files changed, 5 insertions(+), 0 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/cloudstack/blob/8d0bea99/server/src/com/cloud/vm/VirtualMachineManagerImpl.java ---------------------------------------------------------------------- diff --git a/server/src/com/cloud/vm/VirtualMachineManagerImpl.java b/server/src/com/cloud/vm/VirtualMachineManagerImpl.java index 4072531..a53e380 100755 --- a/server/src/com/cloud/vm/VirtualMachineManagerImpl.java +++ b/server/src/com/cloud/vm/VirtualMachineManagerImpl.java @@ -866,6 +866,11 @@ public class VirtualMachineManagerImpl extends ManagerBase implements VirtualMac } } + if (startedVm == null) { + throw new CloudRuntimeException("Unable to start instance '" + vm.getHostName() + + "' (" + vm.getUuid() + "), see management server log for details"); + } + return startedVm; }