Skip to content

Conversation

@bazel-io
Copy link
Member

  • The cache was always written to, even if not enabled.
  • Google RBE doesn't accept Commands without the (deprecated) Platform field set. We set it both on Command and Action, just to be safe.

Fixes #28294 (comment)

Closes #28295.

PiperOrigin-RevId: 856169835
Change-Id: I2479119a173e325a7d39643a36536569f5f831fc

Commit a994609

* The cache was always written to, even if not enabled.
* Google RBE doesn't accept `Command`s without the (deprecated) `Platform` field set. We set it both on `Command` and `Action`, just to be safe.

Fixes bazelbuild#28294 (comment)

Closes bazelbuild#28295.

PiperOrigin-RevId: 856169835
Change-Id: I2479119a173e325a7d39643a36536569f5f831fc
@bazel-io bazel-io added the team-Remote-Exec Issues and PRs for the Execution (Remote) team label Jan 14, 2026
@bazel-io bazel-io requested a review from a team as a code owner January 14, 2026 13:33
@bazel-io bazel-io added the awaiting-review PR is awaiting review from an assigned reviewer label Jan 14, 2026
@bazel-io bazel-io requested a review from meteorcloudy January 14, 2026 13:33
@meteorcloudy meteorcloudy changed the title [9.0.0] Fix remote repo contents cache issues (https://github.com/bazelbuild/bazel/pull/28295) [9.0.0] Fix remote repo contents cache issues (#28295) Jan 14, 2026
Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces two important fixes for the remote repository contents cache. First, it ensures that the cache is only written to when the feature is explicitly enabled by checking the filesystem type of the repository directory. This prevents unintended cache writes. Second, it addresses a compatibility issue with some remote execution backends by setting the Platform field on both Command and Action objects for synthetic actions. The changes are correct and well-targeted to resolve these issues.

@meteorcloudy meteorcloudy added this pull request to the merge queue Jan 14, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 14, 2026
@meteorcloudy meteorcloudy merged commit 728b42b into bazelbuild:release-9.0.0 Jan 14, 2026
46 checks passed
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Jan 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

team-Remote-Exec Issues and PRs for the Execution (Remote) team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants