Merge ~twom/launchpad:add-dev-celery-startup-script into launchpad:master

Proposed by Tom Wardill
Status: Merged
Approved by: Tom Wardill
Approved revision: 33f5b96cc9e1e4c2d49046540dbbf30314ccdb2d
Merge reported by: Otto Co-Pilot
Merged at revision: not available
Proposed branch: ~twom/launchpad:add-dev-celery-startup-script
Merge into: launchpad:master
Diff against target: 13 lines (+1/-1)
1 file modified
utilities/manage-celery-workers.sh (+1/-1)
Reviewer Review Type Date Requested Status
Colin Watson (community) Approve
Review via email: mp+402339@code.launchpad.net

Commit message

Don't error when there's no workers to stop

To post a comment you must log in.
Revision history for this message
Colin Watson (cjwatson) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/utilities/manage-celery-workers.sh b/utilities/manage-celery-workers.sh
index 23293b2..f83b14f 100755
--- a/utilities/manage-celery-workers.sh
+++ b/utilities/manage-celery-workers.sh
@@ -21,7 +21,7 @@ start_worker() {
21stop_worker() {21stop_worker() {
22 queue=$122 queue=$1
23 echo "Stopping worker for $queue"23 echo "Stopping worker for $queue"
24 start-stop-daemon --stop --pidfile "/var/tmp/celeryd-$queue.pid"24 start-stop-daemon --oknodo --stop --pidfile "/var/tmp/celeryd-$queue.pid"
25}25}
2626
27case "$1" in27case "$1" in

Subscribers

People subscribed via source and target branches

to status/vote changes: