Skip to content

fix: auto-detect system architecture instead of hardcoding#8708

Merged
cheeseandcereal merged 1 commit intoeksctl-io:mainfrom
cloudsmithy:main
Apr 29, 2026
Merged

fix: auto-detect system architecture instead of hardcoding#8708
cheeseandcereal merged 1 commit intoeksctl-io:mainfrom
cloudsmithy:main

Conversation

@cloudsmithy
Copy link
Copy Markdown
Contributor

Replace hardcoded ARCH=amd64 with dynamic architecture detection
using uname -m, improving cross-platform compatibility.

  • x86_64amd64
  • aarch64arm64 (Linux ARM)
  • arm64 passes through as-is (macOS Apple Silicon)

This removes the need for users to manually edit the ARCH variable
based on their system.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 2, 2026

Hello cloudsmithy 👋 Thank you for opening a Pull Request in eksctl project. The team will review the Pull Request and aim to respond within 1-10 business days. Meanwhile, please read about the Contribution and Code of Conduct guidelines here. You can find out more information about eksctl on our website

@cloudsmithy
Copy link
Copy Markdown
Contributor Author

Hi team, just a gentle ping on this PR. It improves cross-platform compatibility for the setup script. Could you please help to run the CI checks when you have a moment? Thanks!

@cheeseandcereal cheeseandcereal added kind/docs User documentation labels Apr 29, 2026
@cheeseandcereal cheeseandcereal enabled auto-merge (squash) April 29, 2026 17:54
Copy link
Copy Markdown
Member

@cheeseandcereal cheeseandcereal left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks for the contribution!

@cheeseandcereal cheeseandcereal merged commit dd817c8 into eksctl-io:main Apr 29, 2026
9 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/docs User documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants