Merge pull request #6778 from dokku/6771-handle-directory-removal

Popd out of tmp directory after a git push-based deployment
This commit is contained in:
Jose Diaz-Gonzalez
2024-03-28 22:34:34 -04:00
committed by GitHub
2 changed files with 3 additions and 2 deletions

View File

@@ -15,7 +15,7 @@ git_build_app_repo() {
# clean up after ourselves
local GIT_BUILD_APP_REPO_TMP_WORK_DIR=$(mktemp -d "/tmp/dokku-${DOKKU_PID}-${FUNCNAME[0]}.XXXXXX")
trap "rm -rf '$GIT_BUILD_APP_REPO_TMP_WORK_DIR' >/dev/null" RETURN INT TERM EXIT
trap "popd &>/dev/null || true; rm -rf '$GIT_BUILD_APP_REPO_TMP_WORK_DIR' >/dev/null" RETURN INT TERM EXIT
chmod 755 "$GIT_BUILD_APP_REPO_TMP_WORK_DIR"
unset GIT_DIR GIT_QUARANTINE_PATH GIT_WORK_TREE

View File

@@ -86,10 +86,11 @@ fn-scheduler-deploy-process() {
local CONTAINER_IDX_OFFSET=$((PROC_COUNT + 1))
fi
local container_state_filetype
pushd "$DOKKU_ROOT/$APP" >/dev/null
for container_state_filetype in CONTAINER IP PORT; do
cd "$DOKKU_ROOT/$APP"
find . -maxdepth 1 -name "$container_state_filetype.$PROC_TYPE.*" -printf "%f\n" | sort -t . -k 3 -n | tail -n +$CONTAINER_IDX_OFFSET | xargs rm -f
done
popd &>/dev/null || pushd "/tmp" >/dev/null
}
main() {