Merge ~twom/launchpad:tasks-some-speedy-some-slow into launchpad:master
Proposed by
Tom Wardill
Status: | Merged |
---|---|
Approved by: | Tom Wardill |
Approved revision: | fb58fd0d723fd4d6b2de36c7e789dd7e337d9b70 |
Merge reported by: | Otto Co-Pilot |
Merged at revision: | not available |
Proposed branch: | ~twom/launchpad:tasks-some-speedy-some-slow |
Merge into: | launchpad:master |
Diff against target: |
119 lines (+59/-4) 4 files modified
lib/lp/services/job/celeryconfig.py (+1/-3) lib/lp/services/job/celeryjob.py (+4/-0) lib/lp/services/job/runner.py (+7/-1) lib/lp/services/job/tests/test_celery.py (+47/-0) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Colin Watson (community) | Approve | ||
Review via email: mp+402919@code.launchpad.net |
Commit message
Enable slow lane fallback in celery
Description of the change
* Use the correct key
* Don't re-queue the same task _and_ the slow lane task
* Add a test to ensure we're queueing correctly
To post a comment you must log in.
This currently breaks a lot of celery related job tests:
lp.oci. tests.test_ ocirecipebuildj ob.TestOCIRegis tryUploadJobVia Celery. test_run_ upload
ValueError: expected txn status 'Active' or 'Doomed', but it's 'Committed'