X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=sliver_libvirt.py;h=8189b192c7f67ae6d96c6db6f77d0c1d364b3e4a;hb=e57432c1dfdfeaa52cc32799e2abbc34b7704ce9;hp=4eb86af7dd12606303e356b3f9a8f6747c987fae;hpb=84936c9dc941fc82cd58be637a2d9344b5d6c3c7;p=nodemanager.git

diff --git a/sliver_libvirt.py b/sliver_libvirt.py
index 4eb86af..8189b19 100644
--- 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