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

Enable linux/arm64 docker build and publish #2601

Merged
merged 6 commits into from
Jul 24, 2023

Conversation

dee-kryvenko
Copy link
Contributor

Hey all - I am not sure if it was intentionally not enabled, but on the off chance it wasn't...

@dee-kryvenko dee-kryvenko requested a review from a team as a code owner May 14, 2023 11:46
@deviantintegral
Copy link

I'm prototyping https://github.com/actions/actions-runner-controller by pointing it to my local colima instance on my Mac. I was surprised to see everything running through emulation and not native binaries. It looks like the underlying issue is that the runner is only amd64, so all binaries inside of it fall back to that too.

@ajschmidt8
Copy link
Contributor

ajschmidt8 commented Jul 13, 2023

I hope this gets merged soon. My team is currently building our own arm64 images for our use case. It would be nice if these were published from the source repository.

@nikola-jokic, @TingluoHuang, any thoughts on when/if this PR can be merged?

@nikola-jokic
Copy link
Contributor

Hey everyone,

We are planning to address this issue, hopefully before the next release ☺️

However, we will create a separate PR to address this issue since we also need to publish images with correct labels and have it be part of the release process

…/enable-arm64-docker

Signed-off-by: Dee Kryvenko <dm.kryvenko@gmail.com>
nikola-jokic
nikola-jokic previously approved these changes Jul 21, 2023
Copy link
Contributor

@nikola-jokic nikola-jokic left a comment

Choose a reason for hiding this comment

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

LGTM! @TingluoHuang I tested it on my local repo and just moved these changes there. Everything works 😄 Do you mind also taking a look just in case. I'm confident this is all we need

images/Dockerfile Outdated Show resolved Hide resolved
@TingluoHuang TingluoHuang merged commit a61d3f3 into actions:main Jul 24, 2023
10 checks passed
ashb pushed a commit to ashb/runner that referenced this pull request Jul 25, 2023
* Enable linux/arm64 docker build and publish

* Fix Dockerfile for multi-arch

* Fix Dockerfile - missing args

* Fix Dockerfile - Docker likes x86_64

* Remove unused `RUNNER_ARCH`

---------

Signed-off-by: Dee Kryvenko <dm.kryvenko@gmail.com>
@wyatterp365
Copy link

Details

  • @g😆

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.

None yet

6 participants