Skip to content

ci: modify actlize 0.8.0 ci gate - #29

Open
renyaodanjun wants to merge 1 commit into
v0.8.0from
ci/actlize-0.8.0-gates
Open

renyaodanjun wants to merge 1 commit into
v0.8.0from
ci/actlize-0.8.0-gates

Conversation

@renyaodanjun

Copy link
Copy Markdown
Collaborator

refs: #26

Copilot AI balanced review requested due to automatic review settings October 29, 2026 11:04

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

The workflow does not handle review-submission events, leaving all three gated PPU test jobs skipped after approval timeouts until a manual rerun or another triggering PR event.

1 open issue
What changed in this PR

Updates actlize 0.8.0 CI to conserve PPU testing resources by requiring sequential review gates, addressing the CI sequencing and runner changes in issue #26.

Changes:

  • Requires precheck, AI review, and human approval before unit and integration tests.
  • Replaces the OAM-810E runner label with ZW-810E.
  • Switches PPU execution to t-head/ci-infra/cluster-run@main.
File Description
.github/​workflows/​pr-actlize-test-ppu.yaml Updates default runner labels and the PPU execution action.
.github/​workflows/​ci-actlize.yaml Sequences review gates before testing and updates the unit-test runner.

Review effort: Balanced

actlize-ut-tests-ppu:
name: actlize 0.8.0 UT Tests PPU
needs: precheck
needs: human-review
@renyaodanjun
renyaodanjun force-pushed the ci/actlize-0.8.0-gates branch from 9a9dc2c to 48078b6 Compare October 29, 2026 13:22
Copilot AI balanced review requested due to automatic review settings October 29, 2026 13:22

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

The workflow cannot resume PPU tests after late reviews and removes unrelated FlashAttention regression coverage instead of the requested DeepGEMM job.

2 open issues

Review effort: Balanced

uses: t-head/FlashMLA-for-sail/.github/workflows/flashmla-ut-test-ppu.yml@main
with:
source_mode: 'actlize'
secrets: inherit
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.

4 participants