[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Xen-devel] [PATCH] Fix xm start -c / --vncviewer


  • To: xen-devel@xxxxxxxxxxxxxxxxxxx
  • From: Kevin Wolf <kwolf@xxxxxxx>
  • Date: Thu, 25 Sep 2008 13:27:55 +0200
  • Delivery-date: Thu, 25 Sep 2008 04:27:13 -0700
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>

In xm start the --vncviewer option has no effect, instead -c tries to
both connect to the console and start vncviewer. Additionally, to start
vncviewer it uses the domid variable which is only defined a few lines
later. Thus xm start -c doesn't work at all.

This patch fixes both problems.

Signed-off-by: Kevin Wolf <kwolf@xxxxxxx>
Index: xen-unstable.hg/tools/python/xen/xm/main.py
===================================================================
--- xen-unstable.hg.orig/tools/python/xen/xm/main.py
+++ xen-unstable.hg/tools/python/xen/xm/main.py
@@ -1224,9 +1224,6 @@ def xm_start(args):
     if console_autoconnect:
         start_do_console(dom)
 
-    if console_autoconnect:
-        console.runVncViewer(domid, vncviewer_autopass, True)
-
     try:
         if serverType == SERVER_XEN_API:
             server.xenapi.VM.start(get_single_vm(dom), paused)
@@ -1241,6 +1238,10 @@ def xm_start(args):
     if domid == -1:
         raise xmlrpclib.Fault(0, "Domain '%s' is not started" % dom)
 
+    if vncviewer:
+        console.runVncViewer(domid, vncviewer_autopass, True)
+
+
 def xm_delete(args):
     arg_check(args, "delete", 1)
     dom = args[0]
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.