Skip to content

EXPERIMENT X4: split rbe-toolchain discovery from nativelink rebuild (Tom #2) - #12

Closed
b7r6 wants to merge 2 commits into
mainfrom
ci/x4-rbe-list-split
Closed

b7r6 wants to merge 2 commits into
mainfrom
ci/x4-rbe-list-split

Conversation

@b7r6

@b7r6 b7r6 commented Oct 2, 2026

Copy link
Copy Markdown
Owner

Non-blocking CI experiment. generate-rbe-commands rebuilt nativelink just to list lanes (~124s). This adds a list-only nix target with no nativelink dep. Measuring the critical-path cut on the Nix workflow. Do not merge; does not touch the hard-deck PR #6.

palfrey and others added 2 commits October 2, 2026 13:12
…d (Tom #2)

generate-rbe-commands ran 'nix run .#rbe-toolchain-with-nativelink-test list'
purely to enumerate lanes, which rebuilt nativelink (~124s on the fork) even
though the list subcommand only prints static keys. Add a list-only target with
no nativelink dependency and point discovery at it, removing a full nativelink
rebuild from the critical path before the matrix fan-out.

Co-Authored-By: Claude Opus 4.8 <[email protected]>
@github-actions

github-actions Bot commented Oct 2, 2026

Copy link
Copy Markdown

Some of the pull request description still needs filling in:

  • What and why is missing. Please keep the template's headings.
  • How was this verified? is missing. Please keep the template's headings.
  • Risk is missing. Please keep the template's headings.
  • AI assistance is missing. Please keep the template's headings.

Edit the description and this check re-runs on its own. The sections exist because they are the parts a reviewer cannot get from the diff: why the change is needed, how you know it works, what breaks if it is wrong, and which AI tools helped ("None" is a complete answer).

@b7r6

b7r6 commented Oct 3, 2026

Copy link
Copy Markdown
Owner Author

CI-spike measurement/experiment scratch — closing to keep the queue clean. Superseded by the single clean deliverable #25; branch retained if ever needed.

@b7r6 b7r6 closed this Oct 3, 2026
@b7r6
b7r6 deleted the ci/x4-rbe-list-split 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.

2 participants