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
Arm64 images #1418
Arm64 images #1418
Conversation
a4c1fcd
to
8a45a15
Compare
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.
Just 1 nit and one question. Otherwise this is good.
.github/workflows/ci.yaml
Outdated
@@ -364,15 +364,15 @@ jobs: | |||
run: | | |||
pip install pyyaml | |||
pip install ccm | |||
|
|||
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.
nit:
would be nice to not have these single space character additions as changes.
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.
definitely, sorry about that
.github/workflows/ci.yaml
Outdated
@@ -457,7 +457,7 @@ jobs: | |||
- name: Compute version number | |||
run: | | |||
VERSION=$(printf 'VER\t${project.version}' | mvn help:evaluate | grep '^VER' | cut -f2) | |||
if [ "${GITHUB_REF}" = "refs/heads/master" ] | |||
if [ "${GITHUB_REF}" = "refs/heads/arm64-images" ] |
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.
question:
Shouldn't this be reverted to master
once this PR is merged?
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.
good catch, let me change it before merging
6a718dc
to
1b41172
Compare
1b41172
to
ccd7400
Compare
Fixes #1407
Multi-arch Images built with this PR can be seen on Docker Hub.