Skip to content

Dockerize#5

Open
anieo wants to merge 8 commits into
mainfrom
dockerize
Open

Dockerize#5
anieo wants to merge 8 commits into
mainfrom
dockerize

Conversation

@anieo

@anieo anieo commented May 15, 2026

Copy link
Copy Markdown

No description provided.

anieo and others added 8 commits May 14, 2026 22:21
The branch parameter was only used for the build display name, which made
it misleading. Add a checkout-app stage that uses params.branch to switch
the actual checkout, and move IMAGE_TAG computation after checkout so it
reflects the requested branch.
The pinned versions were upstream release numbers, not Debian package
versions, so apt couldn't resolve any of them and the build failed.
The base image tag (python:3.12-slim-bookworm) already pins the Debian
release; that's the meaningful pin. Also drop python3-dev — it would
install Python 3.11 headers into a 3.12 image.
The +cpu local-version wheels aren't published on PyPI, only on
download.pytorch.org/whl/cpu, so the build failed to resolve
torch==2.4.1+cpu. Also COPY requirements.txt before pip install -r,
which was missing entirely.
COPY commands ran before WORKDIR, so files landed in / instead of /app.
Uvicorn then couldn't import main from /app — "Error loading ASGI app.
Could not import module main".
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants