Skip to content

ci: run linux64_nowallet jobs on ARM runners#7199

Closed
thepastaclaw wants to merge 1 commit into
dashpay:developfrom
thepastaclaw:ci/nowallet-arm
Closed

ci: run linux64_nowallet jobs on ARM runners#7199
thepastaclaw wants to merge 1 commit into
dashpay:developfrom
thepastaclaw:ci/nowallet-arm

Conversation

@thepastaclaw

Copy link
Copy Markdown

Summary

Move the linux64_nowallet depends, build, and test jobs to ARM runners, matching the pattern already used by linux64_multiprocess and linux64_tsan.

Changes

In build.yml, switch runs-on for three jobs from RUNNER_AMD64 to RUNNER_ARM64:

  • depends-linux64_nowallet
  • src-linux64_nowallet
  • test-linux64_nowallet

@github-actions

github-actions Bot commented Mar 3, 2026

Copy link
Copy Markdown

✅ No Merge Conflicts Detected

This PR currently has no conflicts with other open PRs.

@github-actions

Copy link
Copy Markdown

This pull request has conflicts, please rebase.

@thepastaclaw

thepastaclaw commented Mar 30, 2026

Copy link
Copy Markdown
Author

Review Gate

Commit: 35dfbf93

  • Debounce: 10466m ago (need 30m)

  • CI checks: builds passed, 6/6 tests passed

  • CodeRabbit review: no comment yet

  • Off-peak hours: peak window (5am-11am PT) — currently 09:45 AM PT Tuesday

  • Run review now (check to override)

@github-actions

github-actions Bot commented Apr 8, 2026

Copy link
Copy Markdown

This pull request has conflicts, please rebase.

@thepastaclaw

thepastaclaw commented Apr 21, 2026

Copy link
Copy Markdown
Author

Closing as superseded by #7232.

The broader ARM-runner migration in #7232 explicitly keeps linux64_nowallet on x86 as the remaining native x86_64 coverage lane, so this narrower change no longer matches the intended CI layout.

No branch refresh/rebase is needed here: the current head is already cleanly mergeable, but the design direction moved on.

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.

1 participant