-
Notifications
You must be signed in to change notification settings - Fork 407
update ci tools to publish arm64 docker images #1636
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
base: main
Are you sure you want to change the base?
Conversation
9c899c5
to
99fb11f
Compare
looks good so far, I did some restructuring in #1635 feel free to incorporate those (or not :) ) |
@@ -123,10 +123,27 @@ jobs: | |||
env: | |||
GH_TOKEN: ${{ secrets.GITHUB_TOKEN }} | |||
|
|||
generate-matrix-for-build: |
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.
In build job, we need to run docker build and test for each (target, platform) pair. In push job, we need to run build for each target with multiple targets.
@Emilgardis Thanks. I looked into your PR, but it looked hard to merge it to mine. |
@Emilgardis Any update on this? |
Yes, it looks good! but I've not taken the time to test it myself yet. |
Update CI and related tools to publish multi-arch images
Ref: #1597, #1630