From 67085653aed1e70ea94a11680ff39dc11c93b2f8 Mon Sep 17 00:00:00 2001 From: Matt Scott Date: Fri, 24 Nov 2023 07:33:58 -0500 Subject: [PATCH 1/2] Tested the proposed modification to the Docker healthcheck command to support subdirectory root paths to no avail. Staging changes until a proper resolution is proposed. --- docker/Dockerfile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docker/Dockerfile b/docker/Dockerfile index 55ccdfd..3226fc8 100644 --- a/docker/Dockerfile +++ b/docker/Dockerfile @@ -92,6 +92,6 @@ RUN chown ${USER}:${USER} ./configs /app && \ EXPOSE 80/tcp USER ${USER} -HEALTHCHECK CMD ["wget","--output-document=-","--quiet","--tries=1","http://127.0.0.1/"] +HEALTHCHECK --interval=5s --timeout=5s --start-period=20s --retries=5 CMD ["wget","--output-document=-","--quiet","--tries=1","http://127.0.0.1${SCRIPT_NAME:-/}"] ENTRYPOINT ["entrypoint.sh"] CMD ["gunicorn","powerdnsadmin:create_app()"] From 9415b4663f187073372f45201d61c86a0a5d3a34 Mon Sep 17 00:00:00 2001 From: Matt Scott Date: Mon, 11 Dec 2023 08:36:37 -0500 Subject: [PATCH 2/2] Applied the latest suggestion from @ashneilson which appears to be a good fix this time. --- docker/Dockerfile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docker/Dockerfile b/docker/Dockerfile index 3226fc8..519e077 100644 --- a/docker/Dockerfile +++ b/docker/Dockerfile @@ -92,6 +92,6 @@ RUN chown ${USER}:${USER} ./configs /app && \ EXPOSE 80/tcp USER ${USER} -HEALTHCHECK --interval=5s --timeout=5s --start-period=20s --retries=5 CMD ["wget","--output-document=-","--quiet","--tries=1","http://127.0.0.1${SCRIPT_NAME:-/}"] +HEALTHCHECK --interval=5s --timeout=5s --start-period=20s --retries=5 CMD wget --output-document=- --quiet --tries=1 http://127.0.0.1${SCRIPT_NAME:-/} ENTRYPOINT ["entrypoint.sh"] CMD ["gunicorn","powerdnsadmin:create_app()"]