mirror of
https://github.com/dokku/dokku.git
synced 2025-12-29 00:25:08 +01:00
feat: warn when publish ports if scaling up or zero downtime is enabled
Closes #4662
This commit is contained in:
@@ -50,6 +50,31 @@ fn-scheduler-deploy-process() {
|
||||
PARALLEL_DEPLOY_COUNT="$(plugn trigger "app-json-process-deploy-parallelism" "$APP" "$PROC_TYPE")"
|
||||
DOKKU_CHECKS_DISABLED="$DOKKU_CHECKS_DISABLED" INJECT_INIT_FLAG="$INJECT_INIT_FLAG" parallel --will-cite --halt soon,fail=1 --jobs "$PARALLEL_DEPLOY_COUNT" --ungroup <"$PROCESS_TMP_FILE"
|
||||
|
||||
local DOCKER_ARGS
|
||||
DOCKER_ARGS=$(: | plugn trigger docker-args-deploy "$APP" "$IMAGE_TAG" "$PROC_TYPE")
|
||||
DOCKER_ARGS+=$(: | plugn trigger docker-args-process-deploy "$APP" "$IMAGE_SOURCE_TYPE" "$IMAGE_TAG" "$PROC_TYPE")
|
||||
DOCKER_ARGS=" $DOCKER_ARGS "
|
||||
declare -a ARG_ARRAY
|
||||
eval "ARG_ARRAY=($DOCKER_ARGS)"
|
||||
|
||||
local port_published=false
|
||||
for arg in "${ARG_ARRAY[@]}"; do
|
||||
if [[ "$arg" == "-p" ]] || [[ "$arg" == "--publish" ]] || [[ "$arg" == "-P" ]] || [[ "$arg" == "--publish-all" ]]; then
|
||||
port_published=true
|
||||
break
|
||||
fi
|
||||
done
|
||||
|
||||
local warned_on_publish=false
|
||||
if [[ "$port_published" == "true" ]] && [[ "$PROC_COUNT" -gt 1 ]]; then
|
||||
warned_on_publish=true
|
||||
dokku_log_warn "Port publishing may not work as expected with multiple containers. Consider scaling process type $PROC_TYPE to 1."
|
||||
fi
|
||||
if [[ "$port_published" == "true" ]] && [[ "$DOKKU_CHECKS_DISABLED" != "true" ]] && [[ "$warned_on_publish" == "false" ]]; then
|
||||
warned_on_publish=true
|
||||
dokku_log_warn "Port publishing may not work as expected with zero downtime. Consider disabling zero downtime for process type $PROC_TYPE."
|
||||
fi
|
||||
|
||||
plugn trigger scheduler-post-deploy-process "$APP" "$PROC_TYPE"
|
||||
|
||||
# cleanup when we scale down
|
||||
|
||||
Reference in New Issue
Block a user