github/dokku
0
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Merge pull request #4651 from dokku/josegonzalez-patch-1

Browse Source 
fix: do not attempt to retire when there was no pre-existing container
This commit is contained in:
Jose Diaz-Gonzalez
2021-06-13 16:29:37 -04:00
committed by GitHub
parent c09444ae39 d666d67877
commit 9c05854928
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
plugins/scheduler-docker-local/scheduler-deploy
View File

@@ -151,9 +151,11 @@ trigger-scheduler-docker-local-scheduler-deploy() {
fi
trap - INT TERM EXIT
# schedule old container for retirement
dokku_log_verbose "Scheduling old container shutdown for $PROC_TYPE.$CONTAINER_INDEX in $WAIT seconds"
plugn trigger scheduler-register-retired "$APP" "$(cat "$DOKKU_CONTAINER_ID_FILE")" "$WAIT"
if [[ -f "$DOKKU_CONTAINER_ID_FILE" ]]; then
# schedule old container for retirement
dokku_log_verbose "Scheduling old container shutdown for $PROC_TYPE.$CONTAINER_INDEX in $WAIT seconds"
plugn trigger scheduler-register-retired "$APP" "$(cat "$DOKKU_CONTAINER_ID_FILE")" "$WAIT"
fi
# now using the new container
[[ -n "$cid" ]] && echo "$cid" >"$DOKKU_CONTAINER_ID_FILE"