launchpad-reviewers team mailing list archive

[Gavin Panella <gavin.panella@xxxxxxxxxxxxx>]

Gavin Panella has proposed merging lp:~allenap/maas/celery-reloading into lp:maas.

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)
Related bugs:
  Bug #1012965 in MAAS: "celeryd not restarted by reloader service when code updates"
  https://bugs.launchpad.net/maas/+bug/1012965

For more details, see:
https://code.launchpad.net/~allenap/maas/celery-reloading/+merge/110357
-- 
https://code.launchpad.net/~allenap/maas/celery-reloading/+merge/110357
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~allenap/maas/celery-reloading into lp:maas.

=== modified file 'services/reloader/run'
--- services/reloader/run	2012-04-20 16:09:23 +0000
+++ services/reloader/run	2012-06-14 15:28:22 +0000
@@ -60,11 +60,14 @@
 
 
 def handle_pserv_change(event):
-    service_dir = "services/pserv"
+    services = "pserv", "celeryd"
     if is_interesting_python_change(event.name):
-        if is_supervised(service_dir):
-            print("<-- {0.pathname} changed; reloading pserv.".format(event))
-            check_call(("svc", "-du", service_dir))
+        for service in services:
+            service_dir = "services/%s" % service
+            if is_supervised(service_dir):
+                print("<-- {0.pathname} changed; reloading {1}.".format(
+                        event, service))
+                check_call(("svc", "-du", service_dir))
 
 
 class TestReloader(unittest.TestCase):