Skip to content

Conversation

@iancha1992
Copy link
Member

Cache evictions encountered during reads of remote files in RemoteActionFileSystem now result in the build being retried when --experimental_remote_cache_eviction_retries is set to a positive value (the default).

This is enabled by implementing the checkForLostInputs method on the RemoteOutputService, building on the refactoring performed in #25396.

Closes #25358.

PiperOrigin-RevId: 736064349
Change-Id: Idc70d55138c3366d22ece7f0579b242b3c217da8

Commit 85d9cc9

@iancha1992 iancha1992 added team-Performance Issues for Performance teams team-Rules-Java Issues for Java rules team-Remote-Exec Issues and PRs for the Execution (Remote) team awaiting-review PR is awaiting review from an assigned reviewer labels Mar 12, 2025
@iancha1992 iancha1992 requested a review from coeuvre March 12, 2025 20:47
@iancha1992 iancha1992 requested a review from a team as a code owner March 12, 2025 20:47
@iancha1992 iancha1992 enabled auto-merge March 12, 2025 20:47
@coeuvre
Copy link
Member

coeuvre commented Mar 13, 2025

Tests are failing. Please take a look.

@fmeum
Copy link
Collaborator

fmeum commented Mar 13, 2025

#25396 needs to be cherry-picked first. I'll do both.

@fmeum
Copy link
Collaborator

fmeum commented Mar 13, 2025

#25545

Cache evictions encountered during reads of remote files in `RemoteActionFileSystem` now result in the build being retried when `--experimental_remote_cache_eviction_retries` is set to a positive value (the default).

This is enabled by implementing the `checkForLostInputs` method on the `RemoteOutputService`,  building on the refactoring performed in bazelbuild#25396.

Closes bazelbuild#25358.

PiperOrigin-RevId: 736064349
Change-Id: Idc70d55138c3366d22ece7f0579b242b3c217da8
@fmeum
Copy link
Collaborator

fmeum commented Mar 14, 2025

@coeuvre Looks good now!

@iancha1992 iancha1992 added this pull request to the merge queue Mar 26, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Mar 26, 2025
@iancha1992 iancha1992 added this pull request to the merge queue Mar 26, 2025
Merged via the queue into bazelbuild:release-8.2.0 with commit 8212fc9 Mar 26, 2025
45 checks passed
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Mar 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

team-Performance Issues for Performance teams team-Remote-Exec Issues and PRs for the Execution (Remote) team team-Rules-Java Issues for Java rules

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants