zuul
/
zuul
Code Issues Proposed changes

Merge "Don't send reports without any job"

This commit is contained in:
Jenkins 2015-06-23 00:11:25 +00:00 committed by Gerrit Code Review
parent 07cc33c85c 88db9cb888
commit fc074e3876
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
zuul/scheduler.py
View File

@ -1521,7 +1521,13 @@ class BasePipelineManager(object):
def _reportItem(self, item):
self.log.debug("Reporting change %s" % item.change)
ret = True # Means error as returned by trigger.report
if self.pipeline.didAllJobsSucceed(item):
if not self.pipeline.getJobs(item):
# We don't send empty reports with +1,
# and the same for -1's (merge failures or transient errors)
# as they cannot be followed by +1's
self.log.debug("No jobs for change %s" % item.change)
actions = []
elif self.pipeline.didAllJobsSucceed(item):
self.log.debug("success %s" % (self.pipeline.success_actions))
actions = self.pipeline.success_actions
item.setReportedResult('SUCCESS')