use a slightly cleaner Xen API in listvms

[invirt/packages/invirt-remote.git] / files / usr / sbin / sipb-xen-listvms

diff --git a/files/usr/sbin/sipb-xen-listvms b/files/usr/sbin/sipb-xen-listvms
index b67bf6c..a38c9ae 100755 (executable)
--- a/files/usr/sbin/sipb-xen-listvms
+++ b/files/usr/sbin/sipb-xen-listvms
@@ -3,12 +3,12 @@
 import sys
 import time
 sys.path.insert(0, '/usr/lib/xen-3.1-1/lib/python')
-from xen.lowlevel.xs import xs
+from xen.xend.xenstore import xstransact
 
-trans = xs()
+xst = xstransact.xstransact()
 
 def live_vms():
-    domids = set(trans.ls('', '/local/domain'))
+    domids = set(xst.List('/local/domain'))
     domids.discard('0')
 
     vms = dict()
@@ -16,22 +16,22 @@ def live_vms():
     for domid in domids:
         try:
             name, data = get_dom(int(domid))
-        except ValueError:
-            next # went down since we started
+        except TypeError:
+            continue # went down since we started
         if name.startswith('d_'):
             name = name[2:]
             vms[name] = data
     return vms
 
 def get_dom(domid):
-    name = trans.read('', '/local/domain/%d/name' % domid)
+    name = xst.Read('/local/domain/%d/name' % domid)
     data = dict()
     data['domid'] = domid
     # presence of a graphical console
-    data['console'] = trans.read('', '/local/domain/%d/device/vfb/0/state' % domid)
+    data['console'] = xst.Read('/local/domain/%d/device/vfb/0/state' % domid)
     # uptime
-    data['vm'] = trans.read('', '/local/domain/%d/vm' % domid)
-    data['start_time'] = float(trans.read('', '%s/start_time' % data['vm']))
+    data['vm'] = xst.Read('/local/domain/%d/vm' % domid)
+    data['start_time'] = float(xst.Read('%s/start_time' % data['vm']))
     data['uptime'] = time.time()-data['start_time']
     
     return name, data