X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=accounts.py;h=74afcffe90d06e0138d503d2017cccd8755c64bf;hb=c6259a1349497252fac6ba0bbe96512ccd47738b;hp=a9461d1d143375b4c7a88fbcf41f96aebd8ebad0;hpb=208ac1682093530725f297ace0f36f90584d9c57;p=nodemanager.git

diff --git a/accounts.py b/accounts.py
index a9461d1..74afcff 100644
--- a/accounts.py
+++ b/accounts.py
@@ -1,5 +1,4 @@
-# $Id$
-# $URL$
+### 
 
 """Functionality common to all account classes.
 
@@ -142,9 +141,24 @@ If still valid, check if running and configure/start if not."""
             finally: create_sem.release()
         if not isinstance(self._acct, next_class): self._acct = next_class(rec)
         logger.verbose("accounts.ensure_created: %s, running=%r"%(self.name,self.is_running()))
-        if not self.is_running() or next_class != curr_class:
-            self.start(rec)
-        else: self._acct.configure(rec)
+
+        # reservation_alive is set on reervable nodes, and its value is a boolean
+        if 'reservation_alive' in rec:
+            # reservable nodes
+            if rec['reservation_alive']:
+                # this sliver has the lease, it is safe to start it
+                if not self.is_running(): self.start(rec)
+                else: self.configure(rec)
+            else:
+                # not having the lease, do not start it
+                self.configure(rec)
+        # usual nodes - preserve old code
+        # xxx it's not clear what to do when a sliver changes type/class
+        # in a reservable node
+        else:
+            if not self.is_running() or next_class != curr_class:
+                self.start(rec)
+            else: self.configure(rec)
 
     def ensure_destroyed(self): self._destroy(self._get_class())
 
@@ -152,6 +166,9 @@ If still valid, check if running and configure/start if not."""
         self._acct.configure(rec)
         self._acct.start(delay=d)
 
+    def configure(self, rec):
+        self._acct.configure(rec)
+
     def stop(self): self._acct.stop()
 
     def is_running(self):