Skip to content

CI: disable all jobs, add A5 runner bring-up smoke job - #821

Open
luohuan19 wants to merge 2 commits into
hw-native-sys:mainfrom
luohuan19:ci/disable-all-jobs-add-a5-runner-bring-up-smoke-job
Open

CI: disable all jobs, add A5 runner bring-up smoke job#821
luohuan19 wants to merge 2 commits into
hw-native-sys:mainfrom
luohuan19:ci/disable-all-jobs-add-a5-runner-bring-up-smoke-job

Conversation

@luohuan19

Copy link
Copy Markdown
Contributor

Summary

A new A5 server has been added as a CI runner; this gates the workflow down to a single job that only proves the runner can be picked up.

  • Disable every existing job in ci.yml (pre-commit, unit-tests, detect-changes, sim, a2a3, serving-qwen, serving-deepseek) with if: false, kept as false && <original expression> where a condition already existed, so the original gating logic survives verbatim and re-enabling is a prefix removal.
  • Add a5-runner-smoke, structurally a copy of daily_ci.yml's model-tests-a5: same [self-hosted, linux, arm64, npu-a5] labels, 120 min timeout, checkout working directory and env block, and the same step sequence (sparse-fetch action defs, setup-ci-job with needs-device: 'true', kill-orphaned-tasks, build_output cleanup, results upload).
  • Replace that job's Run V4-Pro tests (a5, via task-submit) body with echo "Do something", keeping the step name and its PYPTO_* / PTO2_RING_* env.
  • Use a dedicated a5-smoke build namespace so it does not share the daily job's a5-daily artifacts, and set if-no-files-found: ignore on the upload since the smoke step emits no results.tsv.

@coderabbitai

coderabbitai Bot commented Jul 23, 2026

Copy link
Copy Markdown

Review Change Stack

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 84740c7b-a411-48cb-936c-1e16689715af

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
📝 Walkthrough

Walkthrough

The CI workflow disables existing validation and serving jobs, leaving a new npu-a5 runner smoke job that performs device setup, a placeholder task, cleanup, and optional results artifact upload.

Changes

CI runner smoke workflow

Layer / File(s) Summary
Disable existing CI jobs
.github/workflows/ci.yml
Pre-commit, unit tests, change detection, simulation, A2A3, Qwen serving, and DeepSeek serving jobs are disabled.
Add A5 runner smoke job
.github/workflows/ci.yml
Adds an npu-a5 smoke job with device setup, placeholder execution, cleanup, and tolerant results upload.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

Poem

A bunny hops where A5 wakes,
While old CI paths take brief breaks.
Setup, smoke, cleanup in flight,
Results may land by morning light.
“Do something!” thumps the hare—
A tiny test now waits there.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes disabling CI jobs and adding the A5 runner smoke job.
Description check ✅ Passed The description clearly matches the workflow changes and smoke-job setup described in the PR.
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.

@luohuan19
luohuan19 force-pushed the ci/disable-all-jobs-add-a5-runner-bring-up-smoke-job branch 6 times, most recently from 914b02c to e3850a0 Compare July 23, 2026 07:40
A new A5 server has been added as a CI runner; this gates the workflow
down to a single job that only proves the runner can be picked up.

- Disable every existing job with `if: false` (kept as `false && <original
  expression>` where a condition already existed), so the original gating
  logic survives verbatim and re-enabling is a prefix removal
- Add `a5-runner-smoke`, structurally a copy of daily_ci.yml's
  `model-tests-a5` (npu-a5 runner labels, setup-ci-job with
  needs-device:true, kill-orphaned-tasks, build_output cleanup, results
  upload), with the V4-Pro sweep body replaced by `echo "Do something"`
- Give it its own `a5-smoke` build namespace so it does not share the
  daily job's `a5-daily` artifacts, and set `if-no-files-found: ignore`
  on the upload since the smoke step emits no results.tsv
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