Merge lp:~fginther/helipad/fix_lp1210366 into lp:helipad/ci

[Francis Ginther]

Default the daily release builds to Published only if the job was SUCCESS.

Default the daily release builds to Published only if the job was SUCCESS.

I was able to verify this on my staging server. W/o this change the bad build showed as 'Published', with this applied, it was 'Failed'. There is no further detail available from these builds.

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:41
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https://code.launchpad.net/~fginther/helipad/fix_lp1210366/+merge/180269/+edit-commit-message

http://s-jenkins:8080/job/helipad-ci-ci/67/
Executed test runs:

Click here to trigger a rebuild:
http://s-jenkins:8080/job/helipad-ci-ci/67/rebuild

review: Needs Fixing (continuous-integration)

[Chris Johnston (cjohnston)]

review: Approve

[PS Jenkins bot (ps-jenkins)]

review: Approve (continuous-integration)

=== modified file 'util/heuristics.py'
--- util/heuristics.py	2013-07-31 22:07:43 +0000
+++ util/heuristics.py	2013-08-14 23:47:22 +0000
@@ -20,7 +20,12 @@
     """Determines the published status given a build instance of an
     integration job"""
     build_set = build.downstream_builds.all()
-    status = 'Published'
+
+    # Set a sane default status
+    if build.result.name == 'SUCCESS':
+        status = 'Published'
+    else:
+        status = 'Failed'
 
     # Set status according to the last failed build in the workflow,
     # assumes that the builds are organized in the workflow order.