Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[7.2.0] Don't upload remote input to remote cache #21851

Merged
merged 4 commits into from
Apr 2, 2024

Conversation

iancha1992
Copy link
Member

and when it's missing, treat it as remote cache eviction.

Also revert the workaround for #19513.

Fixes #21777.
Potential fix for #21626 and #21778.

Closes #21825.

PiperOrigin-RevId: 619877088
Change-Id: Ib1204de8440b780e5a6ee6a563a87da08f196ca5

Commit eda0fe4

and when it's missing, treat it as remote cache eviction.

Also revert the workaround for bazelbuild#19513.

Fixes bazelbuild#21777.
Potential fix for bazelbuild#21626 and bazelbuild#21778.

Closes bazelbuild#21825.

PiperOrigin-RevId: 619877088
Change-Id: Ib1204de8440b780e5a6ee6a563a87da08f196ca5
@iancha1992 iancha1992 added team-Remote-Exec Issues and PRs for the Execution (Remote) team awaiting-review PR is awaiting review from an assigned reviewer labels Mar 28, 2024
@iancha1992 iancha1992 requested a review from coeuvre March 28, 2024 20:28
@iancha1992 iancha1992 requested a review from a team as a code owner March 28, 2024 20:28
@iancha1992 iancha1992 enabled auto-merge (squash) March 28, 2024 20:33
@iancha1992 iancha1992 changed the title Don't upload remote input to remote cache [7.2.0] Don't upload remote input to remote cache Mar 28, 2024
@iancha1992 iancha1992 merged commit 2859473 into bazelbuild:release-7.2.0 Apr 2, 2024
32 checks passed
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Apr 2, 2024
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.

2 participants