Skip to content

TIMING AFTER (code-only): heavy fan-out on overhaul base - #20

Closed
b7r6 wants to merge 1 commit into
ci/overhaul-v2-prbasefrom
ci/timing-after-codeonly
Closed

b7r6 wants to merge 1 commit into
ci/overhaul-v2-prbasefrom
ci/timing-after-codeonly

Conversation

@b7r6

@b7r6 b7r6 commented Oct 2, 2026 •

Copy link
Copy Markdown
Owner

What and why

Fork-only CI timing probe (do not merge). A heavy, big-dependency-cone change (edit to the foundational nativelink-util crate, which invalidates a large downstream compile cone) used to measure the cold whole-PR critical path and per-lane breakdown.

How was this verified?

This PR exists to be measured by CI itself; the job durations are the artifact. Compared against its sibling timing PR under identical cache conditions.

Risk

None. Fork-only measurement branch, never merged. The source change is a two-line comment in nativelink-util/src/lib.rs.

AI assistance

An agent (Claude Opus 4.8) created the probe and ran the measurement under human direction.

@github-actions

This comment has been minimized.

@b7r6
b7r6 force-pushed the ci/overhaul-v2-prbase branch from 19b8628 to 3788c2c Compare October 2, 2026 21:38
@b7r6
b7r6 force-pushed the ci/timing-after-codeonly branch from c9dd9ed to e472428 Compare October 2, 2026 21:38
@b7r6

b7r6 commented Oct 3, 2026

Copy link
Copy Markdown
Owner Author

Closing fork-only measurement/experiment PR; superseded by the single clean PR on ci/bazel-dev-warming (#25).

@b7r6 b7r6 closed this Oct 3, 2026
@b7r6
b7r6 deleted the ci/timing-after-codeonly branch October 15, 2026 11:54
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.

1 participant