summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRichard Ipsum <richard.ipsum@codethink.co.uk>2014-06-04 12:44:30 +0100
committerRichard Ipsum <richard.ipsum@codethink.co.uk>2014-06-04 12:44:30 +0100
commitb9e6fad1b63db722fd5413931667542c202e5800 (patch)
tree51ace7478a3b17db71e19629e827f5567983dd6a
parent08eb7f3650acaa20c90bd720b945f3a74459a44d (diff)
downloadmorph-b9e6fad1b63db722fd5413931667542c202e5800.tar.gz
Add comment to explain the use of _JobFailed event
-rw-r--r--distbuild/worker_build_scheduler.py6
1 files changed, 6 insertions, 0 deletions
diff --git a/distbuild/worker_build_scheduler.py b/distbuild/worker_build_scheduler.py
index 99724e29..8caf5511 100644
--- a/distbuild/worker_build_scheduler.py
+++ b/distbuild/worker_build_scheduler.py
@@ -599,6 +599,12 @@ class WorkerConnection(distbuild.StateMachine):
'Failed to populate artifact cache: %s %s' %
(event.msg['status'], event.msg['body']))
+ # We will attempt to remove this job twice
+ # unless we mark it as failed before the BuildController
+ # processes the WorkerBuildFailed event.
+ #
+ # The BuildController will not try to cancel jobs that have
+ # been marked as failed.
self.mainloop.queue_event(WorkerConnection,
_JobFailed(self._job))