Hi,
My colleague happened to find that when installing a Guest,
we cannot access to it by VNC because of the missing
parameter "--vnclisten=0.0.0.0" in virt-install.
Here is the patch, please comment! :)
Signed-off-by: Gu Yanhua <guyanhua-fnst@xxxxxxxxxxxxxx>
---
client/virt/libvirt_vm.py | 8 ++++++++
1 files changed, 8 insertions(+), 0 deletions(-)
diff --git a/client/virt/libvirt_vm.py b/client/virt/libvirt_vm.py
index 65db338..4684aeb 100644
--- a/client/virt/libvirt_vm.py
+++ b/client/virt/libvirt_vm.py
@@ -287,6 +287,7 @@ class VM(virt_vm.BaseVM):
self.serial_console = None
self.redirs = {}
self.vnc_port = 5900
+ self.vnclisten = "0.0.0.0"
self.pci_assignable = None
self.netdev_id = []
self.device_id = []
@@ -298,6 +299,7 @@ class VM(virt_vm.BaseVM):
self.params = params
self.root_dir = root_dir
self.address_cache = address_cache
+ self.vnclisten = "0.0.0.0"
# For now, libvirt does not have a monitor property.
self.monitor = None
self.driver_type = params.get("driver_type", self.LIBVIRT_DEFAULT)
@@ -489,6 +491,9 @@ class VM(virt_vm.BaseVM):
def add_vnc(help, vnc_port):
return " --vnc --vncport=%d" % (vnc_port)
+
+ def add_vnclisten(help, vnclisten):
+ return " --vnclisten=%s " % (vnclisten)
def add_sdl(help):
if has_option(help, "sdl"):
@@ -629,6 +634,9 @@ class VM(virt_vm.BaseVM):
if params.get("vnc_port"):
vm.vnc_port = int(params.get("vnc_port"))
virt_install_cmd += add_vnc(help, vm.vnc_port)
+ if params.get("vnclisten"):
+ vm.vnclisten = params.get("vnclisten")
+ virt_install_cmd += add_vnclisten(help, vm.vnclisten)
elif params.get("display") == "sdl":
virt_install_cmd += add_sdl(help)
elif params.get("display") == "nographic":
-- 1.7.1
--
Best Regards,
Tang chen
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
