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

Specify image arch's to build #348

Merged
merged 2 commits into from
Aug 19, 2022
Merged

Specify image arch's to build #348

merged 2 commits into from
Aug 19, 2022

Conversation

lryanuk
Copy link
Contributor

@lryanuk lryanuk commented Jul 26, 2022

Specify image arch's to include support for linux/arm64

Added using example documented here: https://github.com/docker/build-push-action/blob/master/docs/advanced/multi-platform.md

Specify image arch's to include support for linux/arm64
@@ -36,6 +36,7 @@ jobs:
uses: docker/build-push-action@ad44023a93711e3deb337508980b4b5e9bcdc5dc
with:
context: .
platforms: linux/amd64,linux/arm64
Copy link
Owner

Choose a reason for hiding this comment

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

I'm pretty certain this will also require

 -
        name: Set up QEMU
        uses: docker/setup-qemu-action@v2
-
        name: Set up Docker Buildx
        uses: docker/setup-buildx-action@v2

Can you please also do a test run of the pipeline in your repository and link to the successful result?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added the parts I missed, here's the successful build: https://github.com/lryanuk/chiadog/actions/runs/2883959326

Ran the container on a Raspberry Pi4, seems to be working as expected

Copy link
Owner

Choose a reason for hiding this comment

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

That's great, thanks! 🙌

@lryanuk lryanuk requested a review from martomi August 18, 2022 17:05
@martomi martomi merged commit 26e2951 into martomi:main Aug 19, 2022
@lryanuk lryanuk deleted the patch-1 branch September 1, 2022 16:18
jinnatar pushed a commit to jinnatar/chiadog that referenced this pull request Feb 5, 2023
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