Skip to content

fix(rbe): pin BuildBuddy RBE to managed executors - #88

Merged
Sam Gammon (sgammon) merged 1 commit into
mainfrom
fix/rbe-managed-executors
Oct 18, 2026
Merged

Sam Gammon (sgammon) merged 1 commit into
mainfrom
fix/rbe-managed-executors

Conversation

@sgammon

@sgammon Sam Gammon (sgammon) commented Oct 18, 2026 •

Copy link
Copy Markdown
Member

The rbe (BuildBuddy) job has failed on every PR since ~Oct 11 with:

No registered executors in pool "" with os "linux" with arch "amd64".

The repo config hasn't changed since it last passed (Sept 18), and rotating BUILDBUDDY_API_KEY didn't help. BuildBuddy's use-self-hosted-executors exec property picks between the org's self-hosted executors and BuildBuddy's managed pool, and its default comes from org settings. With the org defaulting to self-hosted, actions are sent to an empty self-hosted pool.

This sets use-self-hosted-executors: false on both RBE platforms (//tools/platforms and e2e/rbe/tools/platforms) so this repo always uses BuildBuddy's managed executors.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

Both changes directly implement the stated fix, with no blocking issues identified.

No open issues 🎉

What changed in this PR

Pins both RBE platforms to BuildBuddy’s managed executors, avoiding failures caused by an empty self-hosted pool.

Changes:

  • Sets use-self-hosted-executors to "false" in both platforms.
  • Documents why the organization default must be overridden.
File Description
tools/​platforms/​BUILD.bazel Pins the main RBE platform to managed executors.
e2e/​rbe/​tools/​platforms/​BUILD.bazel Applies the same setting to the end-to-end RBE platform.

Review effort: Balanced

Set use-self-hosted-executors=false on the RBE platforms. Its default comes
from the BuildBuddy org settings; with the org defaulting to self-hosted,
every remote action failed to schedule with 'No registered executors in
pool ""'.
@codspeed

codspeed Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

Merging this PR will not alter performance

⚠️ Unknown Walltime execution environment detected

Using the Walltime instrument on standard Hosted Runners will lead to inconsistent data.

For the most accurate results, we recommend using CodSpeed Macro Runners: bare-metal machines fine-tuned for performance measurement consistency.

✅ 9 untouched benchmarks


Comparing fix/rbe-managed-executors (f145d9d) with main (13539ef)

Open in CodSpeed

@sgammon
Sam Gammon (sgammon) merged commit 0bdd808 into main Oct 18, 2026
17 checks passed
@sgammon
Sam Gammon (sgammon) deleted the fix/rbe-managed-executors branch October 18, 2026 16:18
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