Skip to content

perf(ci): move the three light-run pole jobs to 32 vCPU runners#109695

Merged
steipete merged 2 commits into
mainfrom
perf/ci-pole-runners-32vcpu
Jul 17, 2026
Merged

perf(ci): move the three light-run pole jobs to 32 vCPU runners#109695
steipete merged 2 commits into
mainfrom
perf/ci-pole-runners-32vcpu

Conversation

@steipete

Copy link
Copy Markdown
Contributor

What Problem This Solves

Post-batch job census (runs 29557967446/29557929941): the light-run critical path is now check-lint (186s shard work), check-dependencies (157s), and build-artifacts (73s build + 53s checks), all on 16 vCPU. These three jobs bound the median PR run at ~4.2min of job time.

Why This Change Was Made

All three workloads parallelize with cores — oxlint shard concurrency scales with nproc (CI thresholds: 8+ cpus), knip scans run concurrently, tsdown fans out — so a 32 vCPU class roughly halves wall clock at similar billed core-minutes (same rationale as the earlier 4→16 bump on check-dependencies/check-test-types, which behaved as predicted). Fork PRs and dispatch runs stay on GitHub-hosted runners as before.

User Impact

None at runtime — CI-only. Light-run PR CI median should drop ~1–1.5min once warm.

Evidence

  • node scripts/run-vitest.mjs test/scripts/ci-workflow-guards.test.ts — 75 passed, 1 skipped (runner-size guards updated with the change).
  • This PR's own CI provides the live before/after: compare its check-lint/check-dependencies/build-artifacts durations against the 186s/157s/126s baselines above.
  • git diff --check clean; YAML matrix rows carry site comments for the sizing rationale.

check-lint (~186s oxlint shards), check-dependencies (~157s concurrent
knip), and build-artifacts (~126s tsdown+checks) are the critical path of
every light PR run; all three parallelize with cores at similar billed
core-minutes. Guard test updated.
@openclaw-barnacle openclaw-barnacle Bot added size: XS maintainer Maintainer-authored PR labels Jul 17, 2026
@clawsweeper clawsweeper Bot added rating: 🦪 silver shellfish Thin PR readiness signal; proof, validation, or implementation needs work. status: 📣 needs proof The PR needs real behavior proof before ClawSweeper can clear the contributor ask. P3 Low-priority cleanup, docs, polish, ergonomics, or speculative work. labels Jul 17, 2026
@steipete
steipete merged commit 3795290 into main Jul 17, 2026
174 of 176 checks passed
@steipete
steipete deleted the perf/ci-pole-runners-32vcpu branch July 17, 2026 06:25
@steipete

Copy link
Copy Markdown
Contributor Author

Merged via squash.

github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request Jul 18, 2026
…claw#109695)

* perf(ci): move the three light-run pole jobs to 32 vCPU runners

check-lint (~186s oxlint shards), check-dependencies (~157s concurrent
knip), and build-artifacts (~126s tsdown+checks) are the critical path of
every light PR run; all three parallelize with cores at similar billed
core-minutes. Guard test updated.

* test: align prerelease-plan guard with the 32 vCPU check-dependencies row
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintainer Maintainer-authored PR P3 Low-priority cleanup, docs, polish, ergonomics, or speculative work. rating: 🦪 silver shellfish Thin PR readiness signal; proof, validation, or implementation needs work. size: XS status: 📣 needs proof The PR needs real behavior proof before ClawSweeper can clear the contributor ask.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant