Launchpad itself

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

  1. Git
  2. lp:~twom/launchpad
  3. add-dev-celery-startup-script
  4. Merge into 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
1diff --git a/utilities/manage-celery-workers.sh b/utilities/manage-celery-workers.sh
2index 23293b2..f83b14f 100755
3--- a/utilities/manage-celery-workers.sh
4+++ b/utilities/manage-celery-workers.sh
5@@ -21,7 +21,7 @@ start_worker() {
6 stop_worker() {
7 queue=$1
8 echo "Stopping worker for $queue"
9- start-stop-daemon --stop --pidfile "/var/tmp/celeryd-$queue.pid"
10+ start-stop-daemon --oknodo --stop --pidfile "/var/tmp/celeryd-$queue.pid"
11 }
12
13 case "$1" in

Subscribers

People subscribed via source and target branches

to status/vote changes: