darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-08-24 16:23:44 -04:00
Code Issues Packages Projects Releases Wiki Activity

Use continue instead of break on SIGKILL

Browse Source 
`break`ing would keep trying to kill the same process and never move to
the next
This commit is contained in:
Athanasios Voutsadakis
2018-02-28 14:50:23 +00:00
parent b158c66705
commit 3045f6233b
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
scripts/run_multi_worker_app_paas.sh
View File

@@ -61,7 +61,7 @@ function on_exit {
if [[ "$n" -ge "$TERMINATE_TIMEOUT" ]]; then if [[ "$n" -ge "$TERMINATE_TIMEOUT" ]]; then
echo "Timeout reached, killing process with pid ${APP_PID}" echo "Timeout reached, killing process with pid ${APP_PID}"
kill -9 ${APP_PID} || true kill -9 ${APP_PID} || true
break continue
else else
echo "Timeout not reached yet, checking " ${APP_PID} echo "Timeout not reached yet, checking " ${APP_PID}
# else, if process is still running send SIGTERM # else, if process is still running send SIGTERM