Skip to content

fix: select released CLI asset by runner platform#3252

Merged
tklauser merged 1 commit into
cilium:mainfrom
immanuwell:fix/action-release-asset-platform
Jun 16, 2026
Merged

fix: select released CLI asset by runner platform#3252
tklauser merged 1 commit into
cilium:mainfrom
immanuwell:fix/action-release-asset-platform

Conversation

@immanuwell

Copy link
Copy Markdown
Contributor

The release install step hardcodes cilium-linux-amd64.tar.gz.
On real hosted runners like ubuntu-24.04-arm, that pulls the wrong binary, so cilium ends up x86-64 and then it kinda trips with Exec format error.

This picks the release archive from runner.os and runner.arch, and fails fast for unsupported combos.

Repro

  1. Run this action on ubuntu-24.04-arm with release-version: v0.19.4.
  2. It downloads cilium-linux-amd64.tar.gz right now.
  3. file /usr/local/bin/cilium shows x86-64, not ARM64.
  4. cilium version --client cant run on that runner.

@immanuwell immanuwell requested a review from a team as a code owner June 12, 2026 10:41
@immanuwell immanuwell requested a review from asauber June 12, 2026 10:41
@maintainer-s-little-helper

Copy link
Copy Markdown

Commit 4032611 does not match "Signed-off-by".

Please follow instructions provided in https://docs.cilium.io/en/stable/contributing/development/contributing_guide/#developer-s-certificate-of-origin

Signed-off-by: immanuwell <pchpr.00@list.ru>
@tklauser tklauser merged commit 77d9a99 into cilium:main Jun 16, 2026
8 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants