Skip to content

Enable linux/arm64 docker build and publish#2601

Merged
TingluoHuang merged 6 commits into
actions:mainfrom
dee-kryvenko:feature/enable-arm64-docker
Jul 24, 2023
Merged

Enable linux/arm64 docker build and publish#2601
TingluoHuang merged 6 commits into
actions:mainfrom
dee-kryvenko:feature/enable-arm64-docker

Conversation

@dee-kryvenko

Copy link
Copy Markdown
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
Copy Markdown

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

ajschmidt8 commented Jul 13, 2023

Copy link
Copy Markdown
Contributor

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
Copy Markdown
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

@nikola-jokic nikola-jokic left a comment

Copy link
Copy Markdown
Contributor

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

Comment thread images/Dockerfile Outdated
@TingluoHuang TingluoHuang merged commit a61d3f3 into actions:main Jul 24, 2023
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>
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.

5 participants