opendev
/
puppet-zuul
Code Issues Proposed changes

Remove 'restart' for v3 executor 

The proper way to shutdown the executor is to issue a 'stop' and then
wait for the executor to finish its current jobs before issuing a
'start'. Doing 'restart' doesn't ever seem to be the right thing to
do, so prevent folks from doing that.

Change-Id: I2630306a7290092c3e59f4be01a82950927bf60b
This commit is contained in:
David Shrewsbury 2018-02-07 07:54:12 -05:00
parent 5c759a128d
commit 9c2fcbd8a4
1 changed files with 1 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
files/zuul-executor.init
View File

@ -116,18 +116,8 @@ case "$1" in
do_reload
log_end_msg $?
;;
restart)
#
# If the "reload" option is implemented then remove the
# 'force-reload' alias
#
log_daemon_msg "Restarting $DESC" "$NAME"
do_stop
do_start
;;
*)
#echo "Usage: $SCRIPTNAME {start|stop|restart|reload|force-reload}" >&2
echo "Usage: $SCRIPTNAME {start|stop|status|restart|force-reload}" >&2
echo "Usage: $SCRIPTNAME {start|stop|status|force-reload}" >&2
exit 3
;;
esac