cloudstack-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
Subject git commit: Pass on the destination host in StartCommand
Date Fri, 04 Jan 2013 13:37:38 GMT
Updated Branches:
  refs/heads/master 42d9c3fbb -> 7240204a5

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,
do they have to change "vnc_listen" in their qemu.conf libvirt settings.


Branch: refs/heads/master
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/    |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/server/src/com/cloud/vm/ b/server/src/com/cloud/vm/
index 9230f4a..4d94fc5 100755
--- a/server/src/com/cloud/vm/
+++ b/server/src/com/cloud/vm/
@@ -764,7 +764,7 @@ public class VirtualMachineManagerImpl implements VirtualMachineManager,
                     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);

View raw message