Merge ~ilasc/turnip:one-repack-task-at-a-time into turnip:master
Proposed by
Ioana Lasc
Status: | Merged |
---|---|
Approved by: | Ioana Lasc |
Approved revision: | 85c51f08ad5c0bcb918f2b2b10f8659df773c624 |
Merge reported by: | Otto Co-Pilot |
Merged at revision: | not available |
Proposed branch: | ~ilasc/turnip:one-repack-task-at-a-time |
Merge into: | turnip:master |
Diff against target: |
140 lines (+63/-7) 6 files modified
Makefile (+14/-2) charm/turnip-celery/config.yaml (+1/-4) charm/turnip-celery/lib/charms/turnip/celery.py (+8/-0) charm/turnip-celery/templates/logrotate.j2 (+13/-0) charm/turnip-celery/templates/turnip-celery-repack.service.j2 (+26/-0) charm/turnip-celery/templates/turnip-celery.service.j2 (+1/-1) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Colin Watson (community) | Approve | ||
Thiago F. Pappacena (community) | Approve | ||
Tom Wardill (community) | Approve | ||
Review via email: mp+402770@code.launchpad.net |
Commit message
Run a dedicated repack celery worker
Description of the change
This proposes we run a dedicated repack worker and limit its concurrency to 1.
I've hardcoded the worker name (repack-worker), dedicated queue name (repacks), concurrency (1) and prefetch-multiplier (1) for it.
For the default worker I've specified the default queue (celery) everything else comes on this queue.
All of the above can be of course turned into configurable items.
Local end to end test results with the 2 workers and queues documented here: https:/
To post a comment you must log in.
Looks decent, just some naming and a missing argument I think.