bug 925285: Set buildbot RETRY for Connection to the other side was lost failures. r=dustin,catlee
authorBen Hearsum <bhearsum@mozilla.com>
Mon, 28 Oct 2013 09:06:19 -0400
changeset 895 b4673f1f2a866400ed43e7c2b14607a610090f62
parent 879 ecfab480076fbc315f535a10d813dfd557384e0f
child 896 12d5a63423f92aee33f03b50a2e1c9296ee78afc
child 901 7189e0a6bead73b39245b883264b29e369800efb
push id662
push userbhearsum@mozilla.com
push dateMon, 28 Oct 2013 13:06:43 +0000
reviewersdustin, catlee
bugs925285
bug 925285: Set buildbot RETRY for Connection to the other side was lost failures. r=dustin,catlee
master/buildbot/process/buildstep.py
--- a/master/buildbot/process/buildstep.py
+++ b/master/buildbot/process/buildstep.py
@@ -861,17 +861,18 @@ class BuildStep:
                 # This should only happen if we've been interrupted
                 assert self.stopped
 
     def finished(self, results):
         if self.stopped:
             # We handle this specially because we don't care about
             # the return code of an interrupted command; we know
             # that this should just be exception due to interrupt
-            results = EXCEPTION
+            if results not in (RETRY, CANCELLED):
+                results = EXCEPTION
             self.step_status.setText(self.describe(True) +
                                  ["interrupted"])
             self.step_status.setText2(["interrupted"])
         if self.progress:
             self.progress.finish()
         self.step_status.stepFinished(results)
         self.releaseLocks()
         self.deferred.callback(results)