Skip to content

[build][bazel] Combine remote setup CI actions#8893

Open
pjreiniger wants to merge 2 commits into
wpilibsuite:mainfrom
bzlmodRio:combine_bazel_remote
Open

[build][bazel] Combine remote setup CI actions#8893
pjreiniger wants to merge 2 commits into
wpilibsuite:mainfrom
bzlmodRio:combine_bazel_remote

Conversation

@pjreiniger
Copy link
Copy Markdown
Contributor

When the bazel remote cache was switched from buildbuddy to a self hosted server in #8342, I asked that the buildbuddy hooks be remain so that I could still use their caching service for local builds.

The downside of this was that my forks builds aren't leveraging buildbuddy, so if I'm fiddling with something heavy like a wpimath robotpy thing, my CI builds never update a cache and never are warm when I push fixups.

This PR combines the setup-bazel-remote and setup-build-buddy actions which set up the bazel remote cache. Rather than having two different version, the correct one will be choosen in the following order:

  1. Use wpi's server with write access if the bazel_remote information is set (This basically would only happen on main branch in wpilibsuite/allwpilib since secrets aren't accessible from builds originating in forks)
  2. Use buildbuddy if the key it is present (This would work for my fork builds)
  3. Fall back to the readonly version of wpi's server

As seen here the build in my fork will run with buildbuddy, and my PR's build here should fall back to readonly mode.

@pjreiniger pjreiniger requested a review from a team as a code owner May 13, 2026 04:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Monorepo organization, Gradle/CMake/Bazel, upstream_utils, etc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants