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

ci: Build Docker for arm64 arch #1423

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mathix420
Copy link

@mathix420 mathix420 commented Aug 24, 2024

  • Trivial upgrades:
    • docker/build-push-action@v5 => v6
  • Added actions:
    • docker/setup-buildx-action@v3
  • Added workflow_dispatch in build-docker.yaml for better DX

For anyone willing to test the arm version ➡️ https://hub.docker.com/r/peachoss/shaka-packager

- Trivial upgrades:
  - docker/build-push-action@v5 => v6
Copy link

google-cla bot commented Aug 24, 2024

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@cosmin
Copy link
Contributor

cosmin commented Sep 10, 2024

@joeyparrish do we need a CLA for a contribution that only touches the github workflows and not the actual code?

- name: Set up Docker Buildx
uses: docker/setup-buildx-action@v3
with:
platforms: linux/amd64,linux/arm64
Copy link
Member

Choose a reason for hiding this comment

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

Does this use qemu for emulation? How slow is it?

Copy link
Author

Choose a reason for hiding this comment

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

Yes it does, and it's quite slow, took 1h on my gh action. Best would be to setup a matrix step on an external runner like you did on other actions but I can't setup this myself.

Copy link
Member

Choose a reason for hiding this comment

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

I don't think I want to add 1h of emulated Docker builds to every PR. Let me try to set it up through our self-hosted runners, at least until GitHub starts offering Arm Linux runners publicly.

@mathix420
Copy link
Author

- name: Set up Docker Buildx
uses: docker/setup-buildx-action@v3
with:
platforms: linux/amd64,linux/arm64
Copy link
Member

Choose a reason for hiding this comment

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

I don't think I want to add 1h of emulated Docker builds to every PR. Let me try to set it up through our self-hosted runners, at least until GitHub starts offering Arm Linux runners publicly.

@cosmin cosmin marked this pull request as draft October 24, 2024 04:51
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.

3 participants