Backout broken patch. production-0.8 FENNEC_27_0b6_BUILD1 FENNEC_27_0b6_RELEASE
authorBen Hearsum <bhearsum@mozilla.com>
Mon, 13 Jan 2014 10:49:13 -0500
branchproduction-0.8
changeset 945 0f30aed72d6f041d2bd1a968a123b8c12d31cea0
parent 943 82bc00fa8694f3afd27941f78d06abe6f88f961a (current diff)
parent 944 ac68b3b6eebbce4256266297385a6de0bbc54dfd (diff)
child 947 1f098fad8ecc1ec706502309900b06f3ed2f7266
push id710
push userbhearsum@mozilla.com
push dateMon, 13 Jan 2014 15:49:19 +0000
Backout broken patch.
--- a/master/buildbot/master.py
+++ b/master/buildbot/master.py
@@ -294,17 +294,21 @@ class BotMaster(service.MultiService):
         d.addCallback(_add)
         d.addCallback(lambda ign: self._updateAllSlaves())
         return d
 
     def _updateAllSlaves(self):
         """Notify all buildslaves about changes in their Builders."""
         dl = []
         for s in self.slaves.values():
-            d = defer.maybeDeferred(s.updateSlave())
+            try:
+                d = s.updateSlave()
+            except Exception:
+                log.err("Failed to update slave %s" % s.slavename)
+                raise
             d.addErrback(log.err)
             dl.append(d)
         return defer.DeferredList(dl)
 
     def shouldMergeRequests(self, builder, req1, req2):
         """Determine whether two BuildRequests should be merged for
         the given builder.