Skip to content

Migrate RBE configuration to dynamic RBE toolchain generator#29647

Closed
coeuvre wants to merge 1 commit into
bazelbuild:masterfrom
coeuvre:push-ulxwtzyxlmwv
Closed

Migrate RBE configuration to dynamic RBE toolchain generator#29647
coeuvre wants to merge 1 commit into
bazelbuild:masterfrom
coeuvre:push-ulxwtzyxlmwv

Conversation

@coeuvre
Copy link
Copy Markdown
Member

@coeuvre coeuvre commented May 26, 2026

This migration replaces the legacy 'rbe_preconfig' rule with the new dynamic RBE toolchain generator ('rbe_config') provided by the unreleased 'bazel_ci_rules' v2.0.0 module extension.

We are dogfooding the unreleased 'bazel_ci_rules' from bazelbuild/continuous-integration master branch (integrated via git_override) to test and validate the dynamic toolchain generation rule under live conditions for both the Bazel repository builds and our downstream CI pipelines. This ensures the dynamic generator is fully robust across major Bazel versions (including Bazel 9.x/master and 8.x) before its official release.

Disabling more tests around sandbox since they are currently broken on RBE.

@coeuvre coeuvre force-pushed the push-ulxwtzyxlmwv branch 2 times, most recently from d3edb3f to e9efdd7 Compare May 26, 2026 11:47
This migration replaces the legacy 'rbe_preconfig' rule with the new
dynamic RBE toolchain generator ('rbe_config') provided by the unreleased
'bazel_ci_rules' v2.0.0 module extension.

We are dogfooding the unreleased 'bazel_ci_rules' master branch (integrated
via git_override) to test and validate the dynamic toolchain generation rule
under live conditions for both the Bazel repository builds and our downstream
CI pipelines. This ensures the dynamic generator is fully robust across major
Bazel versions (including Bazel 9.x/master and 8.x) before its official release.

Disabling more tests around sandbox since they are currently broken on
RBE.
@coeuvre coeuvre force-pushed the push-ulxwtzyxlmwv branch from e9efdd7 to 074c4f8 Compare May 26, 2026 11:48
@coeuvre coeuvre marked this pull request as ready for review May 26, 2026 11:52
@coeuvre coeuvre requested review from Wyverald and meisterT May 26, 2026 11:52
@iancha1992 iancha1992 added awaiting-review PR is awaiting review from an assigned reviewer team-OSS Issues for the Bazel OSS team: installation, release processBazel packaging, website awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally and removed awaiting-review PR is awaiting review from an assigned reviewer labels May 26, 2026
@iancha1992
Copy link
Copy Markdown
Member

@Wyverald @coeuvre Does this need to be cherry-picked in Bazel 8/9?

@coeuvre
Copy link
Copy Markdown
Member Author

coeuvre commented May 27, 2026

Commited as c7268b2.

@coeuvre
Copy link
Copy Markdown
Member Author

coeuvre commented May 27, 2026

@iancha1992 Yes, please.

@coeuvre coeuvre closed this May 27, 2026
@coeuvre coeuvre deleted the push-ulxwtzyxlmwv branch May 27, 2026 07:02
@github-actions github-actions Bot removed the awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally label May 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

team-OSS Issues for the Bazel OSS team: installation, release processBazel packaging, website

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants