Merge lp:~michael.nelson/launchpad/499095-build-retry-depwait-stuck into lp:launchpad
Proposed by
Michael Nelson
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Brad Crittenden | ||||
Approved revision: | not available | ||||
Merged at revision: | not available | ||||
Proposed branch: | lp:~michael.nelson/launchpad/499095-build-retry-depwait-stuck | ||||
Merge into: | lp:launchpad | ||||
Diff against target: |
87 lines (+15/-16) 2 files modified
lib/lp/buildmaster/buildergroup.py (+2/-8) lib/lp/soyuz/doc/buildd-slavescanner.txt (+13/-8) |
||||
To merge this branch: | bzr merge lp:~michael.nelson/launchpad/499095-build-retry-depwait-stuck | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Brad Crittenden (community) | code | Approve | |
Review via email: mp+16865@code.launchpad.net |
To post a comment you must log in.
= Summary =
This is a fix for the 2nd cause of bug 499095, where we sometimes get builds on builders with inconsistent states (date_started is not set because the job was already in the RUNNING state).
For details of the cause, see the comments: /bugs.edge. launchpad. net/soyuz/ +bug/499095/ comments/ 4 /bugs.edge. launchpad. net/soyuz/ +bug/499095/ comments/ 11
https:/
https:/
== Proposed fix ==
Ensure that the job status is correctly reset back to WAITING after buildStatus_ GIVENBACK( ) and buildStatus_ BUILDERFAIL( ).
== Pre-implementation notes ==
Worked with wgrant to diagnose and fix this second cause.
We really need to update the bm logs so we can see the relevant info. Much of the bm logging is done as warning/debug which doesn't seem to be logged currently.
== Implementation details ==
Nothing weird.
== Tests ==
bin/test -vvt buildd- slavescanner. txt
== Demo and Q/A ==
We can QA this on dogfood (wgrant already QA'd the core of the fix locally).
= Launchpad lint =
Checking for conflicts. and issues in doctests and templates.
Running jslint, xmllint, pyflakes, and pylint.
Using normal rules.
Linting changed files: buildmaster/ buildergroup. py soyuz/doc/ buildd- slavescanner. txt
lib/lp/
lib/lp/