Skip to content

#314 Fixes: Added multi-architecture Docker build support with uv dependency management #334

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
RaghuveeRR07 wants to merge 8 commits into from
Closed

#314 Fixes: Added multi-architecture Docker build support with uv dependency management #334

Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
f8004bf
updated dockerfile for multiarch image support
RaghuveeRR07 Aug 28, 2025
6d69c83
Merge branch 'exospherehost:main' into main
RaghuveeRR07 Aug 29, 2025
35a1747
Merge branch 'exospherehost:main' into main
RaghuveeRR07 Sep 2, 2025
7aeac51
Updated dockerfile to support multiarch
RaghuveeRR07 Sep 2, 2025
ade1704
updated dockerfile after implementing suggestions
RaghuveeRR07 Sep 2, 2025
543eebc
updated dockerfile to use version ARG
RaghuveeRR07 Sep 2, 2025
8d2935c
updated dockerfile to use digest
RaghuveeRR07 Sep 2, 2025
c822687
updated dockerfile to use manifest digest
RaghuveeRR07 Sep 2, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 7 additions & 2 deletions state-manager/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,10 @@
FROM python:3.12-slim-bookworm
COPY --from=ghcr.io/astral-sh/uv:latest /uv /uvx /bin/
ARG UV_VERSION=0.4.28
ARG UV_DIGEST=sha256:REPLACE_WITH_MANIFEST_DIGEST
FROM ghcr.io/astral-sh/uv:${UV_VERSION}@${UV_DIGEST} AS uv
ARG PYTHON_DIGEST=sha256:REPLACE_WITH_MANIFEST_DIGEST
FROM python:3.12-slim-bookworm@${PYTHON_DIGEST}

COPY --from=uv /uv /usr/local/bin/uv
Comment on lines +1 to +7

@NiveditJain NiveditJain Sep 2, 2025

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@RaghuveeRR07 I do not understand why the earlier implementation was not working but current one is

@NiveditJain NiveditJain Sep 2, 2025

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

can you please explain more ?

@RaghuveeRR07 RaghuveeRR07 Sep 2, 2025

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

The earlier implementation had the latest tag due to which the manifest files for some architecture like armv7 did not exist. this was one of the error I encountered buildx error: “no match for platform in manifest”.

Manifest file is a just a JSON file, which has info about which image is suitable for which arch.

This is the only change I implemented because when I even tried pushing the image of docker hub, armv7 failed. So I understood there is need to specify the version while installing uv.

All the other changes were suggested by this codeReview bot, so I am not really sure what the "digest" tag does.


WORKDIR /api-server

Expand Down