oops, too much messing about

[nodemanager.git] / sliver_libvirt.py

diff --git a/sliver_libvirt.py b/sliver_libvirt.py
index 4eb86af..8189b19 100644 (file)
--- a/sliver_libvirt.py
+++ b/sliver_libvirt.py
@@ -98,6 +98,8 @@ class Sliver_Libvirt(Account):
             output += Sliver_Libvirt.dom_details (dom)
         return output
 
+    # Thierry : I am not quite sure if /etc/libvirt/lxc/<>.xml holds a reliably up-to-date
+    # copy of the sliver XML config; I feel like issuing a virsh dumpxml first might be safer
     def repair_veth(self):
         # See workaround email, 2-14-2014, "libvirt 1.2.1 rollout"
         xml = open("/etc/libvirt/lxc/%s.xml" % self.name).read()
@@ -112,13 +114,14 @@ class Sliver_Libvirt(Account):
         logger.log_call(command)
 
     def start(self, delay=0):
-        ''' Just start the sliver '''
+        '''Just start the sliver'''
         logger.verbose('sliver_libvirt: %s start'%(self.name))
 
         # Check if it's running to avoid throwing an exception if the
-        # domain was already running, create actually means start
+        # domain was already running
         if not self.is_running():
             try:
+                # create actually means start
                 self.dom.create()
             except Exception, e:
                 # XXX smbaker: attempt to resolve slivers that are stuck in