Skip to content
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.

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

Docker multiarch image #703

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

Docker multiarch image #703

wants to merge 13 commits into from

Conversation

eternal-flame-AD
Copy link
Member

@eternal-flame-AD eternal-flame-AD commented Oct 16, 2024

fixes #257 .
fixes #350

Simply make build-docker-multiarch and then docker run should work. I implemented building JS from docker as well but opt-in, just for people who want's to build a usable local container without having to do anything outside of docker.

Current blockers: (nevermind I had a nap and figured out how to do it)

The last step of installing runtime dependency need qemu, it's not easy to get around that unfortunately

Signed-off-by: eternal-flame-AD <yume@yumechi.jp>
Signed-off-by: eternal-flame-AD <yume@yumechi.jp>
Signed-off-by: eternal-flame-AD <yume@yumechi.jp>
@eternal-flame-AD eternal-flame-AD requested a review from a team as a code owner October 16, 2024 00:42
@eternal-flame-AD eternal-flame-AD changed the title Docker multiarch Docker multiarch image Oct 16, 2024
Copy link

codecov bot commented Oct 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.42%. Comparing base (58084c8) to head (eeb2984).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #703      +/-   ##
==========================================
- Coverage   80.46%   79.42%   -1.05%     
==========================================
  Files          56       56              
  Lines        2191     2639     +448     
==========================================
+ Hits         1763     2096     +333     
- Misses        337      452     +115     
  Partials       91       91              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Signed-off-by: eternal-flame-AD <yume@yumechi.jp>
@eternal-flame-AD
Copy link
Member Author

eternal-flame-AD commented Oct 16, 2024

Here's the Docker scout cloud analysis, you should be able to reproduce it with docker scout CLI.

Sorry about the force pushes, messed up a rebase :(

image

image

@eternal-flame-AD eternal-flame-AD force-pushed the docker-multiarch branch 2 times, most recently from 9caea83 to 377ef3c Compare October 16, 2024 05:23
Copy link
Member

@jmattheis jmattheis left a comment

Choose a reason for hiding this comment

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

Nicely done, some remarks in subcomments.

docker/Dockerfile Outdated Show resolved Hide resolved
docker/Dockerfile Outdated Show resolved Hide resolved
docker/Dockerfile Outdated Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
docker/Dockerfile Outdated Show resolved Hide resolved
docker/Dockerfile Outdated Show resolved Hide resolved
Makefile Show resolved Hide resolved
docker/Dockerfile Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
docker/Dockerfile Show resolved Hide resolved
Signed-off-by: eternal-flame-AD <yume@yumechi.jp>
Signed-off-by: eternal-flame-AD <yume@yumechi.jp>
Signed-off-by: eternal-flame-AD <yume@yumechi.jp>
Signed-off-by: eternal-flame-AD <yume@yumechi.jp>
docker/Dockerfile Outdated Show resolved Hide resolved
docker/Dockerfile Outdated Show resolved Hide resolved
docker/Dockerfile Outdated Show resolved Hide resolved
docker/Dockerfile Outdated Show resolved Hide resolved
docker/Dockerfile Outdated Show resolved Hide resolved
Signed-off-by: eternal-flame-AD <yume@yumechi.jp>
Signed-off-by: eternal-flame-AD <yume@yumechi.jp>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Add curl package on ARM Dockerfile Multi-Arch Docker Image
2 participants