Merge pull request sidekiq#1424 from opengovernment/feature/generaliz…

…ed-start-script Starting sidekiq via Capistrano requires a tty and nohup.
luddement · Jan 10, 2014 · c95a6b1 · c95a6b1
2 parents 7e6929d + cac620e
commit c95a6b1
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/examples/chef/cookbooks/sidekiq/templates/default/sidekiq.erb b/examples/chef/cookbooks/sidekiq/templates/default/sidekiq.erb
@@ -159,7 +159,8 @@ if [ -d $APP_ROOT ]; then
         fi
       fi
       if [ ! -f $PID_FILE ]; then
-        sudo -u $USER -H -i sh -c "cd $APP_ROOT; $COMMAND"  >> $LOG_FILE 2>&1 &
+        # http://maymay.net/blog/2010/03/17/how-to-work-around-sorry-you-must-have-a-tty-to-run-sudo-without-sacrificing-security/
+        nohup su - --session-command="cd $APP_ROOT; $COMMAND" $USER >> $LOG_FILE 2>&1 &
         RESULT=$?
         logger -t "monit-sidekiq[$$]" "Started with pid $! and exit $RESULT"
         echo $! > $PID_FILE