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

.github/workflows: Add release support to linux/arm64 platform #541

Merged
merged 2 commits into from
Apr 30, 2024

Conversation

kfirtoledo
Copy link
Collaborator

Add release support to linux/arm64 platform.

Signed-off-by: Kfir Toledo <kfir.toledo@ibm.com>
@kfirtoledo kfirtoledo requested a review from elevran as a code owner April 25, 2024 18:51
@kfirtoledo kfirtoledo requested a review from orozery April 25, 2024 18:52
@@ -32,7 +32,7 @@ jobs:
run: make push-image IMAGE_VERSION=${{ github.ref_name }}
- name: Build and compress binaries
run: |
for pair in "linux:amd64" "darwin:amd64" "darwin:arm64"; do
for pair in "linux:amd64" "linux:arm64" "darwin:amd64" "darwin:arm64"; do
Copy link
Collaborator

Choose a reason for hiding this comment

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

you can have two for loops ( {linux,darwin} x {amd64,arm64} ) instead of having to parse os:arch in the next line

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

For now, I prefer to leave it like this, for case we want support arch only in one os

@kfirtoledo kfirtoledo merged commit 4d51eb6 into clusterlink-net:main Apr 30, 2024
9 checks passed
@kfirtoledo kfirtoledo deleted the release branch April 30, 2024 07:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants