zuul
/
zuul
Code Issues Proposed changes

Fix missing build warning. 

It's okay if an individual queue manager can't find a build, but
we care if _no_ queue managers find the build.

Change-Id: If2470a8bda5316563f92980d631b96037ec76c0f
This commit is contained in:
James E. Blair 2012-05-31 10:03:13 -07:00
parent e9d45c3c13
commit c84dd2666b
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
zuul/scheduler.py
View File

@ -214,6 +214,7 @@ class Scheduler(threading.Thread):
for manager in self.queue_managers.values():
if manager.onBuildCompleted(build):
return
self.log.warning("Build %s not found by any queue manager" % (build))
class BaseQueueManager(object):
@ -287,8 +288,7 @@ for change %s:" % (job, change))
def onBuildCompleted(self, build):
self.log.debug("Build %s completed" % build)
if build not in self.building_jobs:
self.log.warning("Build %s not found (may have been canceled)" % (
build))
self.log.debug("Build %s not found" % (build))
# Or triggered externally, or triggered before zuul started,
# or restarted
return False