Return-Path: X-Original-To: apmail-incubator-cloudstack-commits-archive@minotaur.apache.org Delivered-To: apmail-incubator-cloudstack-commits-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id B26ECD74C for ; Tue, 8 Jan 2013 00:41:23 +0000 (UTC) Received: (qmail 48800 invoked by uid 500); 8 Jan 2013 00:41:19 -0000 Delivered-To: apmail-incubator-cloudstack-commits-archive@incubator.apache.org Received: (qmail 48646 invoked by uid 500); 8 Jan 2013 00:41:19 -0000 Mailing-List: contact cloudstack-commits-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: cloudstack-dev@incubator.apache.org Delivered-To: mailing list cloudstack-commits@incubator.apache.org Received: (qmail 47990 invoked by uid 99); 8 Jan 2013 00:41:19 -0000 Received: from tyr.zones.apache.org (HELO tyr.zones.apache.org) (140.211.11.114) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 08 Jan 2013 00:41:19 +0000 Received: by tyr.zones.apache.org (Postfix, from userid 65534) id BA5E411140; Tue, 8 Jan 2013 00:41:18 +0000 (UTC) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: bfederle@apache.org To: cloudstack-commits@incubator.apache.org X-Mailer: ASF-Git Admin Mailer Subject: [40/54] [abbrv] git commit: Pass on the destination host in StartCommand Message-Id: <20130108004118.BA5E411140@tyr.zones.apache.org> Date: Tue, 8 Jan 2013 00:41:18 +0000 (UTC) Pass on the destination host in StartCommand I added this in commit bc94948e0604e0e5931759be3c3d3155e84686f6 to be able to bind the VNC on KVM on the Private IP Address of the Hypervisor. This got (accidentally) reverted in commit 110903a91a21c04b931a26354a04bd7f534ba050 breaking this behaviour with KVM. By passing the destination host again in StartCommand we are able to bind the VNC to the private IP address of the hypervisor. This makes sure the VNC is not open for the world and users don't have to firewall these ports, nor do they have to change "vnc_listen" in their qemu.conf libvirt settings. Project: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/commit/7240204a Tree: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/tree/7240204a Diff: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/diff/7240204a Branch: refs/heads/ui-quick-view-v2 Commit: 7240204a507cce8143c248e6aa635da6dad60ed0 Parents: 42d9c3f Author: Wido den Hollander Authored: Fri Jan 4 14:31:25 2013 +0100 Committer: Wido den Hollander Committed: Fri Jan 4 14:37:23 2013 +0100 ---------------------------------------------------------------------- .../com/cloud/vm/VirtualMachineManagerImpl.java | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/blob/7240204a/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 9230f4a..4d94fc5 100755 --- a/server/src/com/cloud/vm/VirtualMachineManagerImpl.java +++ b/server/src/com/cloud/vm/VirtualMachineManagerImpl.java @@ -764,7 +764,7 @@ public class VirtualMachineManagerImpl implements VirtualMachineManager, Listene VirtualMachineTO vmTO = hvGuru.implement(vmProfile); cmds = new Commands(OnError.Stop); - cmds.addCommand(new StartCommand(vmTO)); + cmds.addCommand(new StartCommand(vmTO, dest.getHost())); vmGuru.finalizeDeployment(cmds, vmProfile, dest, ctx);