Skip to content

ci: modify actlize 1.0.0 ci gate - #28

Open
renyaodanjun wants to merge 1 commit into
v1.0.0from
ci/actlize-1.0.0-gates
Open

renyaodanjun wants to merge 1 commit into
v1.0.0from
ci/actlize-1.0.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:03

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

Late AI or human reviews cannot restart skipped test jobs because the workflow lacks a review-submission recovery path and compatible guards on all four downstream test jobs.

1 open issue
What changed in this PR

This PR updates actlize 1.0.0 CI toward issue #26’s resource-saving workflow by requiring review gates before PPU tests.

Changes:

  • Sequence precheck, AI review, and human approval before unit and integration tests.
  • Replace the OAM-810E runner label with ZW-810E.
  • Switch 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 tests and updates the unit-test runner matrix.

Review effort: Balanced

actlize-ut-tests-ppu:
name: actlize 1.0.0 UT Tests PPU
needs: precheck
needs: human-review
@renyaodanjun
renyaodanjun force-pushed the ci/actlize-1.0.0-gates branch from 368be02 to e005924 Compare October 29, 2026 13:23
Copilot AI balanced review requested due to automatic review settings October 29, 2026 13:23

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

Late reviews cannot restart timed-out approval gates, leaving PPU tests skipped, and the new FA job still uses the obsolete OAM-810E runner instead of ZW-810E.

2 open issues

Review effort: Balanced

Comment on lines +72 to +73
version: v2.7.2
source_mode: 'actlize1.0.0'
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