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

Multi-arch docker image #197

Merged
merged 2 commits into from
Jul 2, 2024
Merged

Conversation

noamcohen97
Copy link
Contributor

@noamcohen97 noamcohen97 commented Jun 26, 2024

Resolves #81

@noamcohen97
Copy link
Contributor Author

noamcohen97 commented Jun 26, 2024

@scefali, @armenzg Can you please help pushing this change to enable ARM64 support?
This was an issue before, but now that GitHub supports github-hosted ARM64 runners it affects many.

Co-authored-by: Bartosz Bednarek <895578+bbednarek@users.noreply.github.com>
@@ -16,48 +17,59 @@ jobs:
docker-build:
name: Build & pubish Docker images
runs-on: ubuntu-latest
permissions:
packages: write
Copy link
Member

Choose a reason for hiding this comment

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

steps:
- name: Checkout repo
uses: actions/checkout@v3
with:
fetch-depth: 0

- name: Set up QEMU
uses: docker/setup-qemu-action@v3
Copy link
Member

Choose a reason for hiding this comment

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

uses: docker/setup-qemu-action@v3

- name: Set up Docker Buildx
uses: docker/setup-buildx-action@v3
Copy link
Member

Choose a reason for hiding this comment

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

docker push ghcr.io/getsentry/action-release-builder-image:latest
docker push ghcr.io/getsentry/action-release-image:latest
- name: Build and push
uses: docker/build-push-action@v6
Copy link
Member

Choose a reason for hiding this comment

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

- name: Build and push
uses: docker/build-push-action@v6
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.

Before this PR we were only building linux/amd64 but after this PR we will also build linux/arm64

uses: docker/build-push-action@v6
with:
platforms: linux/amd64,linux/arm64
push: ${{ github.ref == 'refs/heads/master' }}
Copy link
Member

Choose a reason for hiding this comment

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

For anyone reading this PR afterwards, we will only push the image when the check runs under the master branch.

@armenzg armenzg merged commit b4057e3 into getsentry:master Jul 2, 2024
0 of 3 checks passed
@noamcohen97 noamcohen97 deleted the multi-platform branch July 2, 2024 14:03
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.

Action fails with ARM architecture
3 participants