Skip to content

CI: skip ut-a5 while the a5 runner is under repair - #1533

Merged
ChaoZheng109 merged 1 commit into
hw-native-sys:mainfrom
ChaoZheng109:ci/skip-ut-a5-runner-repair
Jul 27, 2026
Merged

CI: skip ut-a5 while the a5 runner is under repair#1533
ChaoZheng109 merged 1 commit into
hw-native-sys:mainfrom
ChaoZheng109:ci/skip-ut-a5-runner-repair

Conversation

@ChaoZheng109

Copy link
Copy Markdown
Collaborator

The a5 self-hosted runner is out for repair, so ut-a5 has no host to run on and sits waiting for one instead of reporting a result. This short-circuits its existing gate with false && so it reports skipped, mirroring exactly what #1481 already did for st-onboard-a5 on the same runner.

Nothing declares needs: ut-a5 and main has no required status checks, so the job's absence blocks no merge. Coverage for a5 changes falls back to st-sim-a5, which runs on GitHub-hosted runners.

To re-enable once the runner returns: drop the false && from the if: gate (same as the note left on st-onboard-a5).

The a5 self-hosted runner is out for repair, so ut-a5 has no host to run
on and sits waiting for one instead of reporting a result. Short-
circuiting its existing gate with `false &&` makes it report skipped,
mirroring what hw-native-sys#1481 already did for st-onboard-a5 on the same runner.

Nothing declares `needs: ut-a5` and main has no required status checks,
so the job's absence blocks no merge. Coverage for a5 changes falls back
to st-sim-a5, which runs on GitHub-hosted runners.
@coderabbitai

coderabbitai Bot commented Jul 27, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 75274ec2-6cbf-4827-866c-13d061b4e086

📥 Commits

Reviewing files that changed from the base of the PR and between a53cc16 and 72ff7c1.

📒 Files selected for processing (1)
  • .github/workflows/ci.yml

📝 Walkthrough

Walkthrough

The GitHub Actions ut-a5 job condition now includes an unconditional false && guard, preventing the job from running regardless of the docs_only output.

Changes

CI job gating

Layer / File(s) Summary
Disable ut-a5 execution
.github/workflows/ci.yml
The ut-a5 condition now always evaluates to false before checking docs_only.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Poem

A tiny false guards the gate,
So ut-a5 must now wait.
The rabbit checks the workflow run,
And finds the job has come undone.
“Hopfully next time, little test!”

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: skipping the ut-a5 CI job while the A5 runner is unavailable.
Description check ✅ Passed The description matches the change and explains why ut-a5 is being short-circuited.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@ChaoZheng109
ChaoZheng109 merged commit 1c475a7 into hw-native-sys:main Jul 27, 2026
16 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.

1 participant