Skip to content

Shard Bazel Windows tests across jobs#22408

Merged
starr-openai merged 1 commit into
mainfrom
starr/windows-bazel-pr-job-fanout-20260512
May 13, 2026
Merged

Shard Bazel Windows tests across jobs#22408
starr-openai merged 1 commit into
mainfrom
starr/windows-bazel-pr-job-fanout-20260512

Conversation

@starr-openai
Copy link
Copy Markdown
Contributor

@starr-openai starr-openai commented May 13, 2026

Summary

  • split the single PR-blocking Bazel Windows test leg into four Windows shard jobs
  • preserve the existing required Windows Bazel check name with a lightweight aggregate gate
  • keep Linux/macOS Bazel test jobs and the separate Windows clippy/release jobs unchanged

Why

The ordinary PR Windows Bazel test leg was one GitHub Actions job, so Bazel only had in-job parallelism. This gives that lane real job-level fanout across separate Windows hosts while keeping the target set disjoint via stable label hashing.

Evidence

  • final pre-rebase green run: 25774733562
  • Windows shard target counts: 61/212, 48/212, 52/212, 51/212
  • Windows test fanout completed in about 7m29s versus a recent monolithic median around 22m26s

Notes

  • this is scoped to the Bazel Windows test leg only
  • each shard keeps the existing Windows cross-compile/RBE path and restores the former monolithic Windows test cache
  • shard jobs do not upload duplicate repository caches after test work, keeping cache cleanup off the PR-blocking shard path
  • no local validation run; relying on GitHub Actions for the workflow-shaped check

@starr-openai starr-openai force-pushed the starr/windows-bazel-pr-job-fanout-20260512 branch 3 times, most recently from ced41f9 to 8daec49 Compare May 13, 2026 02:35
Co-authored-by: Codex <noreply@openai.com>
@starr-openai starr-openai force-pushed the starr/windows-bazel-pr-job-fanout-20260512 branch from 8daec49 to 8eefa83 Compare May 13, 2026 18:13
@starr-openai starr-openai marked this pull request as ready for review May 13, 2026 19:15
Copy link
Copy Markdown
Collaborator

@bolinfest bolinfest left a comment

Choose a reason for hiding this comment

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

TYSM!

@starr-openai starr-openai merged commit d666238 into main May 13, 2026
31 checks passed
@starr-openai starr-openai deleted the starr/windows-bazel-pr-job-fanout-20260512 branch May 13, 2026 19:46
@github-actions github-actions Bot locked and limited conversation to collaborators May 13, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants