Skip to content

Fix arm64 CI runner mismatches and stale action version comment#7433

Merged
friedrichg merged 1 commit intomasterfrom
fix-ci-runner-mismatch
Apr 17, 2026
Merged

Fix arm64 CI runner mismatches and stale action version comment#7433
friedrichg merged 1 commit intomasterfrom
fix-ci-runner-mismatch

Conversation

@friedrichg
Copy link
Copy Markdown
Member

What this PR does: Two arm64 integration test matrix entries (integration_overrides, integration_querier_microservices_mode) were using the amd64 runner ubuntu-24.04 instead of ubuntu-24.04-arm, causing them to run on the wrong architecture.

Also fix stale version comment on actions/upload-artifact in scorecards.yml (v3.pre.node20 -> v6.0.0 to match the pinned hash).

Which issue(s) this PR fixes:
Fixes #

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]
  • docs/configuration/v1-guarantees.md updated if this PR introduces experimental flags

Two arm64 integration test matrix entries (integration_overrides,
integration_querier_microservices_mode) were using the amd64 runner
ubuntu-24.04 instead of ubuntu-24.04-arm, causing them to run on
the wrong architecture.

Also fix stale version comment on actions/upload-artifact in
scorecards.yml (v3.pre.node20 -> v6.0.0 to match the pinned hash).

Signed-off-by: Friedrich Gonzalez <1517449+friedrichg@users.noreply.github.com>
@dosubot dosubot bot added the ci/cd label Apr 17, 2026
Copy link
Copy Markdown
Member

@CharlieTLe CharlieTLe left a comment

Choose a reason for hiding this comment

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

Thanks!

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Apr 17, 2026
@friedrichg friedrichg merged commit 31f7ede into master Apr 17, 2026
93 of 95 checks passed
@friedrichg friedrichg deleted the fix-ci-runner-mismatch branch April 17, 2026 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/cd lgtm This PR has been approved by a maintainer size/XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants