commit 972a2e82a316ef2fc042a82d9380d8c2b79b22b2
author Sapan Bhatia <gwsapan@gmail.com> Wed Oct 02 00:03:02 2013 -0400
committer Sapan Bhatia <gwsapan@gmail.com> Wed Oct 02 00:03:02 2013 -0400
tree d35ace08691a80c4148b189990b169f8e56bbfb8
parent 4fa85fb13f41338f263ae0b2b9d39aec87703d8b

Steps not in the dependency graph should also be executed

planetstack/observer/event_loop.py

diff --git a/planetstack/observer/event_loop.py b/planetstack/observer/event_loop.py
index 7ad3efc..55f9a98 100644
--- a/planetstack/observer/event_loop.py
+++ b/planetstack/observer/event_loop.py
@@ -55,6 +55,8 @@
         except KeyError:
             pass
         order.append(n)
+
+    order.extend(set(steps)-set(order))
     return order
 
 class PlanetStackObserver:
@@ -96,7 +98,7 @@
 
         provides_dict = {}
         for s in self.sync_steps:
-            self.step_lookup[s.__name__] = s
+            self.step_lookup[s.__name__] = s 
             for m in s.provides:
                 try:
                     provides_dict[m.__name__].append(s.__name__)
@@ -148,7 +150,7 @@
 
         dependency_graph = step_graph
 
-        self.ordered_steps = toposort(dependency_graph, self.sync_steps)
+        self.ordered_steps = toposort(dependency_graph, map(lambda s:s.__name__,self.sync_steps))
         print "Order of steps=",self.ordered_steps
         self.load_run_times()
         
@@ -197,12 +199,11 @@
     def run(self):
         if not self.driver.enabled or not self.driver.has_openstack:
             return
-
         while True:
             try:
                 logger.info('Waiting for event')
                 tBeforeWait = time.time()
-                self.wait_for_event(timeout=300)
+                self.wait_for_event(timeout=30)
                 logger.info('Observer woke up')
 
                 # Set of whole steps that failed
@@ -216,7 +217,7 @@
                     start_time=time.time()
                     
                     sync_step = step(driver=self.driver)
-                    sync_step.__name__ = step.__name__    
+                    sync_step.__name__ = step.__name__
                     #sync_step.dependencies = self.dependencies[sync_step.name]
                     sync_step.debug_mode = debug_mode
 
@@ -246,6 +247,7 @@
                                 failed_step_objects.extend(failed_objects)
                             self.update_run_time(sync_step)
                         except:
+                            raise
                             failed_steps.append(S)
                 self.save_run_times()
             except: