From 6a6e60836b7ab93b4fdda3a53a7756d2477b4be2 Mon Sep 17 00:00:00 2001 From: Pavan Kumar Date: Sun, 7 Jun 2026 12:43:17 +0530 Subject: [PATCH] fix(docker): drop undefined $PYTHONPATH prefix in production Dockerfiles MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The api and conserver Dockerfiles set PYTHONPATH as `${PYTHONPATH}:/app/...`, but the base image (python:3.12-slim) never sets PYTHONPATH, so the variable was undefined — BuildKit emitted an UndefinedVar warning and the value carried a stray leading colon (an empty/cwd path entry). Drop the prefix. The listed directories are unchanged, and `/app` is already among them, so the implicit current-dir entry was redundant — no behavior change, just a clean PYTHONPATH and no warning. Co-Authored-By: Claude Opus 4.8 (1M context) --- docker/Dockerfile.api | 2 +- docker/Dockerfile.conserver | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/docker/Dockerfile.api b/docker/Dockerfile.api index 8ad8411..ab6601b 100644 --- a/docker/Dockerfile.api +++ b/docker/Dockerfile.api @@ -40,7 +40,7 @@ COPY common/ /app/common/ COPY api/ /app/api/ COPY docker/ /app/docker/ -ENV PYTHONPATH="${PYTHONPATH}:/app/:/app/common/:/app/api/" +ENV PYTHONPATH="/app/:/app/common/:/app/api/" ENTRYPOINT ["/app/docker/wait_for_redis.sh"] CMD ["opentelemetry-instrument", "uvicorn", "api:app", "--host", "0.0.0.0", "--port", "8000"] diff --git a/docker/Dockerfile.conserver b/docker/Dockerfile.conserver index 686f111..20baae2 100644 --- a/docker/Dockerfile.conserver +++ b/docker/Dockerfile.conserver @@ -46,7 +46,7 @@ COPY common/ /app/common/ COPY conserver/ /app/conserver/ COPY docker/ /app/docker/ -ENV PYTHONPATH="${PYTHONPATH}:/app/:/app/common/:/app/conserver/" +ENV PYTHONPATH="/app/:/app/common/:/app/conserver/" ENTRYPOINT ["/app/docker/wait_for_redis.sh"] CMD ["opentelemetry-instrument", "python", "/app/conserver/main.py"]