This repository has been archived by the owner on Jun 13, 2023. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 24
Arm64 amd64 images #52
Closed
arpita0911patel
wants to merge
12
commits into
AlabamaWaterInstitute:main
from
arpita0911patel:arm64_amd64_images
Closed
Changes from all commits
Commits
Show all changes
12 commits
Select commit
Hold shift + click to select a range
162b1b5
Updated CI workflow to add manual build for arm image
arpita0911patel e11b99f
Merge branch 'main' of https://github.com/arpita0911patel/CloudInfra
arpita0911patel bdce97a
Added set up QEMU and buildx
arpita0911patel 67d5319
Seperate arm and x86 manual build flow
arpita0911patel f1e2595
Build image with latest and latest_arm tags
arpita0911patel 9b16d8d
added buildx for multiarch and keeping image tag as latest
arpita0911patel 8a2f2d1
Revert "Added set up QEMU and buildx"
arpita0911patel ecf1731
Revert "Revert "Added set up QEMU and buildx""
arpita0911patel ab7922a
Revert "added buildx for multiarch and keeping image tag as latest"
arpita0911patel 1b7bf37
Revert "Build image with latest and latest_arm tags"
arpita0911patel 1b5346a
Updated repo and tag name
arpita0911patel c3c3f79
Updated CI workflow for main branch (arm image)
arpita0911patel File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -6,10 +6,16 @@ on: | |
- docker/** | ||
jobs: | ||
build-and-push: | ||
runs-on: macos-latest | ||
runs-on: ubuntu-latest | ||
steps: | ||
- name: deploy | ||
uses: actions/checkout@v2 | ||
- name: Set up QEMU | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I don't think buildx is building an actual arm image. Since the |
||
uses: docker/setup-qemu-action@v2 | ||
with: | ||
platforms: 'arm64,arm' | ||
- name: Set up Docker Buildx | ||
uses: docker/setup-buildx-action@v2 | ||
- name: Build and push ARM Docker image for NGEN | ||
uses: ./.github/action_templates/build-and-push | ||
with: | ||
|
2 changes: 1 addition & 1 deletion
2
...ub/workflows/docker_image_main_branch.yml → ...orkflows/docker_image_main_branch_x86.yml
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,21 @@ | ||
name: Manual Build and Push ARM Docker image to Docker Hub | ||
on: | ||
workflow_dispatch: | ||
jobs: | ||
build-and-push: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- name: deploy | ||
uses: actions/checkout@v2 | ||
- name: Set up QEMU | ||
uses: docker/setup-qemu-action@v2 | ||
with: | ||
platforms: 'arm64,arm' | ||
- name: Set up Docker Buildx | ||
uses: docker/setup-buildx-action@v2 | ||
- name: Build and push Docker image | ||
uses: ./.github/action_templates/build-and-push | ||
with: | ||
docker-auth-token: ${{ secrets.DOCKER_AUTH_TOKEN }} | ||
github-sha: ${{ github.sha }} | ||
image-platform: "arm" |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this build step is where you want to use
QEMU
and then provide the--platform
flag to the build so it can build either one...