Skip to content

feat(scripts): add arm64 support for Go and Helm on standalone devices - #352

Closed
nhat-14 wants to merge 0 commit into
margo:developmentfrom
nhat-14:feature/support-arm64-standalone-device
Closed

feat(scripts): add arm64 support for Go and Helm on standalone devices#352
nhat-14 wants to merge 0 commit into
margo:developmentfrom
nhat-14:feature/support-arm64-standalone-device

Conversation

@nhat-14

@nhat-14 nhat-14 commented Jul 21, 2026

Copy link
Copy Markdown

Description

Adds architecture-aware installation for standalone device setup so Go and Helm are installed correctly on both amd64 and arm64 hosts.

  • Added a new resolve_target_arch() helper to detect and normalize host architecture.
  • Updated Go install to download the correct archive using detected architecture.
  • Updated Helm install to build tar name and extraction path dynamically from detected architecture.
  • Removed hardcoded Helm tar assignment from the device-agent setup path.

Motivation

The scripts previously assumed amd64, which could fail on arm64 standalone devices or install incorrect binaries.

Testing

  • Verified Go download URL is architecture-specific.
  • Verified Helm tarball name and extracted directory path match detected architecture.
  • Verified amd64 behavior remains unchanged.

Checklist

  • Tests added/updated
  • Documentation updated
  • No breaking changes (or documented in footer)
  • Conventional commits used

@nhat-14

nhat-14 commented Jul 22, 2026

Copy link
Copy Markdown
Author

@nitparihar
@sanjujunnuthula
This PR was opened from a fork as followed the CONTRIBUTING.md. But it seems to be the reason behind the fail of
sandbox-sanity-test / sandbox-e2e-functional-test-job

Forked one does not expose sandbox repository secrets like HARBOR_PASSWORD to that run and the oras login failure.
Should I make a branch directly from the margo/sandbox

@vireshnavalli

vireshnavalli commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Hi @nhat-14
we will test this and let you know if any changes required. Thanks for contribution.

You can also verify with sanity pipeline using your branch name as in below screen. So make a branch directly from development branch. Also you need to modify runs-on: to ubuntu-24.04-arm64 in pipeline file, by default pipeline currently runs on amd64.

image

@vireshnavalli vireshnavalli added the SPRINT_21 SPRINT_21 label Jul 23, 2026
@vireshnavalli vireshnavalli added this to the SPRINT_21_PR3 milestone Jul 23, 2026
@nhat-14 nhat-14 closed this Jul 24, 2026
@nhat-14
nhat-14 force-pushed the feature/support-arm64-standalone-device branch from dbdff2b to 4a719c1 Compare July 24, 2026 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

SPRINT_21 SPRINT_21

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants