Merge lp://staging/~vila/ubuntu-ci-services-itself/run-worker-conversion into lp://staging/ubuntu-ci-services-itself
Proposed by
Vincent Ladeuil
| Status: | Merged |
|---|---|
| Approved by: | Evan |
| Approved revision: | 349 |
| Merged at revision: | 368 |
| Proposed branch: | lp://staging/~vila/ubuntu-ci-services-itself/run-worker-conversion |
| Merge into: | lp://staging/ubuntu-ci-services-itself |
| Diff against target: |
510 lines (+121/-226) 6 files modified
branch-source-builder/bsbuilder/run_worker.py (+30/-51) ci-utils/ci_utils/amqp_worker.py (+7/-4) ci-utils/ci_utils/tests/test_amqp_worker.py (+8/-5) image-builder/imagebuilder/run_worker.py (+15/-84) run-tests (+1/-1) test_runner/tstrun/run_worker.py (+60/-81) |
| To merge this branch: | bzr merge lp://staging/~vila/ubuntu-ci-services-itself/run-worker-conversion |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Evan (community) | Approve | ||
| PS Jenkins bot (community) | continuous-integration | Approve | |
|
Review via email:
|
|||
Commit message
Convert the run-workers over to the new common run-worker helper
Description of the change
Just a trivial fix on top of Andy's excellent work, only tests were impacted.
https:/
Diff against target: 453 lines (+112/-220) 4 files modified
Can't argue against removing 100 lines ;)
To post a comment you must log in.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
Looks good