Merge lp://staging/~doanac/ubuntu-ci-services-itself/run-worker-conversion into lp://staging/ubuntu-ci-services-itself
Status: | Merged |
---|---|
Merged at revision: | 368 |
Proposed branch: | lp://staging/~doanac/ubuntu-ci-services-itself/run-worker-conversion |
Merge into: | lp://staging/ubuntu-ci-services-itself |
Prerequisite: | lp://staging/~doanac/ubuntu-ci-services-itself/runworker-cancellable |
Diff against target: |
453 lines (+112/-220) 4 files modified
branch-source-builder/bsbuilder/run_worker.py (+30/-51) ci-utils/ci_utils/amqp_worker.py (+7/-4) image-builder/imagebuilder/run_worker.py (+15/-84) test_runner/tstrun/run_worker.py (+60/-81) |
To merge this branch: | bzr merge lp://staging/~doanac/ubuntu-ci-services-itself/run-worker-conversion |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Vincent Ladeuil (community) | Approve | ||
PS Jenkins bot (community) | continuous-integration | Needs Fixing | |
Review via email: mp+210078@code.staging.launchpad.net |
Commit message
convert the run-workers over to the new common run-worker helper
Description of the change
Part 3 of the 3 part series to improve our run-workers and allow them to be cancelled.
This is mostly identation changes and deleted lines, but the diff view is the most friendly. It might be easier to just read each run_worker.py file as a whole.
NOTE: I've removed a long fix-me in the test-runner:
286 - # FIXME: There is a potentially confusing failure mode here: by reusing
287 - # the same testbed for all packages, we may miss bad dependencies for
288 - # one package because that missing dependency is provided by another
289 - # package. It's vague and unclear that we should care about this edge
290 - # case at this point -- vila 2014-02-01
I think this needs to be opened as a bug
PASSED: Continuous integration, rev:346 s-jenkins. ubuntu- ci:8080/ job/uci- engine- ci/349/
http://
Executed test runs:
Click here to trigger a rebuild: s-jenkins. ubuntu- ci:8080/ job/uci- engine- ci/349/ rebuild
http://