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

Support for different architectures #53

Closed
michael-baraboo-cnr opened this issue Jan 10, 2022 · 2 comments · Fixed by #64
Closed

Support for different architectures #53

michael-baraboo-cnr opened this issue Jan 10, 2022 · 2 comments · Fixed by #64
Assignees
Labels
backlog Planned for future

Comments

@michael-baraboo-cnr
Copy link

michael-baraboo-cnr commented Jan 10, 2022

Hello, and thanks for the useful action.

It seems this action currently only supports amd64, which appears to be hardcoded in the get.helm.sh url.
https://github.com/Azure/setup-helm/blob/master/lib/run.js#L59

It would be nice to have this support alternative architectures such as arm64 the same way the azure/setup-kubectl action does.

Currently working around this by copying a lot of the same logic in bash.

             mkdir -p /tmp/helm /opt/hostedtoolcache/helm/3.7.2/arm64/linux-arm64/
             wget https://get.helm.sh/helm-v3.7.2-linux-arm64.tar.gz
             tar -zxvf helm-v3.7.2-linux-arm64.tar.gz -C /tmp/helm
             mv /tmp/helm/linux-arm64/helm /opt/hostedtoolcache/helm/3.7.2/arm64/linux-arm64/
             echo "/opt/hostedtoolcache/helm/3.7.2/arm64/linux-arm64/" >> $GITHUB_PATH
@michael-baraboo-cnr michael-baraboo-cnr added the need-to-triage Requires investigation label Jan 10, 2022
@github-actions
Copy link

This issue is idle because it has been open for 14 days with no activity.

@github-actions github-actions bot added the idle Inactive for 14 days label Jan 24, 2022
@qpetraroia
Copy link
Collaborator

Hey @michael-baraboo-cnr,

For sure, we will look into this! This is in the pipeline!

@qpetraroia qpetraroia added backlog Planned for future and removed need-to-triage Requires investigation idle Inactive for 14 days labels Feb 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog Planned for future
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants