Skip to content

Lock digests and unify uv layers#27

Merged
zvonimir-delas merged 1 commit intomainfrom
optimise-dockerfile
Oct 9, 2025
Merged

Lock digests and unify uv layers#27
zvonimir-delas merged 1 commit intomainfrom
optimise-dockerfile

Conversation

@zvonimir-delas
Copy link
Contributor

  • Locked SHA256 digests to avoid rebuilds and improve reproducibility
    • For version and security updates, change the version and the digest
  • Unified the two uv sync commands so both are before the COPY src - this should minimise rebuilds
    • The second step appears to have been the same as first, except without the --no-install-project flag. But uv has --no-install-project included by default if there is no build-system in the pyproject.toml file. Since this project doesn't have build-system, I don't think the steps were actually any different
  • Removed ENV UV_COMPILE_BYTECODE=1 - this was the step that was timing out. It might result in a slightly slower startup, but on the very first request only. This is a workaround for the server performance issue.

@github-actions
Copy link

github-actions bot commented Oct 9, 2025

New build (#11) deployed to staging 🚧
- Test it on the staging URL: papa.amfws.staging.arg.tech
- If you update the PR or create another one, a new build will be created, replacing this one
- View build log

Note: If this PR is to main, only authorized users will be allowed to merge. Once merged to main, the build will be promoted to the production server. Ensure the staging build works correctly before merging.

@zvonimir-delas zvonimir-delas merged commit 8769001 into main Oct 9, 2025
2 checks passed
@zvonimir-delas zvonimir-delas deleted the optimise-dockerfile branch October 9, 2025 14:21
@github-actions
Copy link

github-actions bot commented Oct 9, 2025

In progress: build being promoted to production 🏗️
- Target server: argand
- View build log

@github-actions
Copy link

github-actions bot commented Oct 9, 2025

Build deployed to production 🏠
- Target server: argand
- Your app's URL: papa.amfws.arg.tech
- Tip: delete the source branch, and for future work create a fresh one from main to avoid merge conflicts
- View build log

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.

1 participant