Skip to content

improve(ci): balance compact whole test jobs#104472

Merged
vincentkoc merged 9 commits into
mainfrom
perf/ci-whole-shard-balance
Jul 11, 2026
Merged

improve(ci): balance compact whole test jobs#104472
vincentkoc merged 9 commits into
mainfrom
perf/ci-whole-shard-balance

Conversation

@vincentkoc

@vincentkoc vincentkoc commented Jul 11, 2026

Copy link
Copy Markdown
Member

Closes #104471

What Problem This Solves

Canonical pull-request CI could wait more than six minutes on one compact Node test job because the serial tooling config was packed into a broad whole-config runner.

Why This Change Was Made

The compact PR planner now balances whole-config groups deterministically and stripes the 329-file serial tooling config into three near-equal groups. The Docker and isolated tooling configs remain explicit whole-config groups with coverage invariants.

This changes only canonical pull-request admission: main pushes, manual dispatches, release validation, tooling-isolated coverage, and tooling-Docker coverage retain their existing topology.

The compact plan grows from 18 to 20 Node jobs. That two-job increase removes the dominant serial critical path while remaining trivial against the live 10,000 runner-registration bucket.

The wider shard coverage also exposed and fixed three latent test couplings: a stale direct-import assertion, a child config probe inheriting the parent shard filter, and a deadline assertion depending on a live millisecond clock.

User Impact

Maintainers and contributors get materially faster pull-request feedback without dropping test coverage or changing release validation behavior.

Evidence

@vincentkoc vincentkoc self-assigned this Jul 11, 2026
@openclaw-barnacle openclaw-barnacle Bot added scripts Repository scripts size: XS maintainer Maintainer-authored PR labels Jul 11, 2026
@vincentkoc
vincentkoc marked this pull request as ready for review July 11, 2026 12:42
@clawsweeper clawsweeper Bot added rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR. P3 Low-priority cleanup, docs, polish, ergonomics, or speculative work. labels Jul 11, 2026
@clawsweeper

clawsweeper Bot commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

Codex review: stale review; fresh review needed.

Summary
The latest durable ClawSweeper review was for head 0322fe1b2bac3c2e53126855c8a1ba938e0a2d0b, but the PR head is now e60eae386a1f2e3bed27522995cf3c5c67e4b9b0. Its old verdict and PR readiness labels are no longer current.

Next step
Run or wait for a fresh ClawSweeper review on the current PR head.

Review history (2 earlier review cycles)
  • reviewed 2026-07-11T12:46:36.906Z sha 59ef0cf :: needs maintainer review before merge. :: none
  • reviewed 2026-07-11T12:53:05.358Z sha 59ef0cf :: needs maintainer review before merge. :: none

@vincentkoc
vincentkoc force-pushed the perf/ci-whole-shard-balance branch from 59ef0cf to 6b8b172 Compare July 11, 2026 12:59
@openclaw-barnacle openclaw-barnacle Bot added docs Improvements or additions to documentation size: S and removed size: XS labels Jul 11, 2026
@clawsweeper clawsweeper Bot removed rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR. labels Jul 11, 2026
@vincentkoc
vincentkoc force-pushed the perf/ci-whole-shard-balance branch from 5e368b0 to 0322fe1 Compare July 11, 2026 13:19
@vincentkoc
vincentkoc merged commit b4c6ee2 into main Jul 11, 2026
111 checks passed
@vincentkoc
vincentkoc deleted the perf/ci-whole-shard-balance branch July 11, 2026 13:29
@vincentkoc

Copy link
Copy Markdown
Member Author

Merged via rebase.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Improvements or additions to documentation maintainer Maintainer-authored PR P3 Low-priority cleanup, docs, polish, ergonomics, or speculative work. scripts Repository scripts size: S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI compact whole shard packing creates a 403s critical path

1 participant