Skip to content

Update docker entrypoint #200

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
ericof wants to merge 2 commits into
base: uv
Choose a base branch
from
Open

Update docker entrypoint #200

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
d07f2c7
Add export to the list of sub-commands available on docker-entrypoint.sh
ericof Feb 6, 2026
7840196
Add instrumentation support to docker-entrypoint.sh
ericof Feb 6, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions news/+exporter.feature
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
Add `export` to the commands available via docker-entrypoint.sh . @ericof
- Usage is ./docker-entrypoint.sh <SITE_ID> <PATH_TO_EXPORT_SITE_DATA>
1 change: 1 addition & 0 deletions news/+instrumentation.feature
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Add support to instrument `runwsgi` by setting the environment variable INSTRUMENTATION_CMD, that should be the name of a command present in the Python virtual environment. @ericof
1 change: 1 addition & 0 deletions news/+zope-ini.feature
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Support providing a different path to zope.ini by setting the environment variable ZOPE_INI_PATH . @ericof
19 changes: 17 additions & 2 deletions skeleton/docker-entrypoint.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,19 @@ else
sudo=""
fi

# Instrumentation support
if [[ -v INSTRUMENTATION_CMD ]] ; then
echo "Enabling Instrumentation using $INSTRUMENTATION_CMD command"
WSGI_CMD=("$VENVBIN/$INSTRUMENTATION_CMD" "$VENVBIN/runwsgi")
else
WSGI_CMD=("$VENVBIN/runwsgi")
fi

# Allow pointing to a different Zope ini path
if [ -z "${ZOPE_INI_PATH}" ]; then
ZOPE_INI_PATH=etc/zope.ini
fi

# MAIN ENV Vars
[ -z ${SECURITY_POLICY_IMPLEMENTATION+x} ] && export SECURITY_POLICY_IMPLEMENTATION=C
[ -z ${VERBOSE_SECURITY+x} ] && export VERBOSE_SECURITY=off
Expand Down Expand Up @@ -145,9 +158,9 @@ if [[ "$1" == "start" ]]; then
if [[ -v LISTEN_PORT ]] ; then
# Ensure the listen port can be set via container --environment.
# Necessary to run multiple backends in a single Podman / Kubernetes pod.
sed -i "s/port = 8080/port = ${LISTEN_PORT}/" etc/zope.ini
sed -i "s/port = 8080/port = ${LISTEN_PORT}/" "${ZOPE_INI_PATH}"
fi
exec $sudo $VENVBIN/runwsgi -v etc/zope.ini config_file=${CONF}
exec $sudo "${WSGI_CMD[@]}" -v "${ZOPE_INI_PATH}" config_file=${CONF}
elif [[ "$1" == "create-classic" ]]; then
export TYPE=classic
exec $sudo $VENVBIN/zconsole run etc/${CONF} /app/scripts/create_site.py
Expand All @@ -159,6 +172,8 @@ elif [[ "$1" == "create-site" ]]; then
exec $sudo $VENVBIN/zconsole run etc/${CONF} /app/scripts/create_site.py
elif [[ "$1" == "console" ]]; then
exec $sudo $VENVBIN/zconsole debug etc/${CONF}
elif [[ "$1" == "export" ]]; then
exec $sudo $VENVBIN/plone-exporter etc/${CONF} "${@:2}" "${@:3}"
elif [[ "$1" == "import" ]]; then
exec $sudo $VENVBIN/plone-importer etc/${CONF} "${@:2}" "${@:3}"
elif [[ "$1" == "run" ]]; then
Expand Down