Skip to content

fix(dockerfile): replace hardcoded path by ARG#444

Merged
jonperron merged 1 commit intoBoavizta:mainfrom
jonperron:fix/dockerfile
Jan 12, 2026
Merged

fix(dockerfile): replace hardcoded path by ARG#444
jonperron merged 1 commit intoBoavizta:mainfrom
jonperron:fix/dockerfile

Conversation

@jonperron
Copy link
Copy Markdown
Collaborator

Docker images built since #442 were broken, because there was a hardcoded path in the Dockerfile. This PR replaces the hardcoded path by the usage of the PY_VERSION, so future updates will be simpler.

make-docker is working

Successfully built 0e7936c9b4b6
Successfully tagged boavizta/boaviztapi:1.3.1

make-docker-build-development is working

Successfully built b4f4b83d677a
Successfully tagged boavizta/boaviztapi:01-11-26

make-docker-run-development is working

docker run -p 5000:5000 boavizta/boaviztapi:01-11-26
INFO:     Started server process [1]
INFO:     Waiting for application startup.
INFO:     Application startup complete.
INFO:     Uvicorn running on http://0.0.0.0:5000 (Press CTRL+C to quit)

docker run is working with production image

docker run -p 5000:5000 boavizta/boaviztapi:1.3.13 
INFO:     Started server process [1]
INFO:     Waiting for application startup.
INFO:     Application startup complete.
INFO:     Uvicorn running on http://0.0.0.0:5000 (Press CTRL+C to quit)
INFO:     172.17.0.1:42648 - "GET / HTTP/1.1" 307 Temporary Redirect
INFO:     172.17.0.1:42648 - "GET // HTTP/1.1" 200 OK
INFO:     172.17.0.1:42648 - "GET /favicon.ico HTTP/1.1" 404 Not Found

Changelog
* Use PY_VERSION in copy instruction
Copy link
Copy Markdown
Collaborator

@Shillaker Shillaker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch 👍

@jonperron jonperron merged commit 2cf475e into Boavizta:main Jan 12, 2026
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