Skip to content

[8.7.0] Support remote cache CDC#28900

Merged
iancha1992 merged 1 commit intobazelbuild:release-8.7.0from
tyler-french:tfrench/cdc-8.6.0
Mar 26, 2026
Merged

[8.7.0] Support remote cache CDC#28900
iancha1992 merged 1 commit intobazelbuild:release-8.7.0from
tyler-french:tfrench/cdc-8.6.0

Conversation

@tyler-french
Copy link
Copy Markdown
Contributor

This PR enables support for content-defined chunking (FastCDC) for large uploads/downloads to remote cache. See #28437 for more details.

RELNOTES[NEW]: Added --experimental_remote_cache_chunking flag to read and write large blobs to/from the remote cache in chunks. Requires server support.

@tyler-french tyler-french requested a review from a team as a code owner March 5, 2026 16:33
@tyler-french
Copy link
Copy Markdown
Contributor Author

FYI @tjgq @fmeum

@tyler-french tyler-french changed the base branch from release-8.6.0 to release-8.7.0 March 5, 2026 21:20
@tyler-french tyler-french changed the title [8.6.0] Support remote cache CDC [8.7.0] Support remote cache CDC Mar 5, 2026
@iancha1992 iancha1992 requested a review from tjgq March 5, 2026 22:26
@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 5, 2026
@iancha1992 iancha1992 added this to the 8.7.0 release blockers milestone Mar 5, 2026
@iancha1992
Copy link
Copy Markdown
Member

@tjgq @fmeum @tyler-french @tjgq Did we even have this change merged to master branch?

@tyler-french
Copy link
Copy Markdown
Contributor Author

@tjgq @fmeum @tyler-french @tjgq Did we even have this change merged to master branch?

Not yet. I created separate PRs since there were merge conflicts. We should wait until the main PR is merged to master first.

@iancha1992
Copy link
Copy Markdown
Member

We'll try and cherry-pick this first via the bot. Thank you for your work on the original PR! We'll try to import it asap.

@iancha1992 iancha1992 closed this Mar 12, 2026
@github-actions github-actions Bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Mar 12, 2026
@iancha1992 iancha1992 reopened this Mar 17, 2026
@github-actions github-actions Bot added the awaiting-review PR is awaiting review from an assigned reviewer label Mar 17, 2026
@iancha1992
Copy link
Copy Markdown
Member

@tyler-french @tjgq Do we need to make changes to this PR as per #28437 (comment)?

@tyler-french
Copy link
Copy Markdown
Contributor Author

@tyler-french @tjgq Do we need to make changes to this PR as per #28437 (comment)?

Yes, working on this now.

@tyler-french tyler-french changed the title [8.7.0] Support remote cache CDC wip [8.7.0] Support remote cache CDC Mar 18, 2026
@tyler-french tyler-french marked this pull request as draft March 18, 2026 08:03
@tyler-french tyler-french changed the title wip [8.7.0] Support remote cache CDC [8.7.0] Support remote cache CDC Mar 18, 2026
@tyler-french tyler-french marked this pull request as ready for review March 18, 2026 16:29
@iancha1992
Copy link
Copy Markdown
Member

@tjgq I think this is ready now. Could you please take a look? Thanks @tyler-french!

@iancha1992 iancha1992 enabled auto-merge March 18, 2026 17:37
@iancha1992 iancha1992 removed this from the 8.7.0 release blockers milestone Mar 18, 2026
@iancha1992 iancha1992 added this pull request to the merge queue Mar 25, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Mar 25, 2026
@tyler-french
Copy link
Copy Markdown
Contributor Author

@iancha1992 can we retry the merge queue?

@iancha1992 iancha1992 added this pull request to the merge queue Mar 25, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Mar 25, 2026
@iancha1992 iancha1992 added this pull request to the merge queue Mar 25, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Mar 25, 2026
@iancha1992 iancha1992 added this pull request to the merge queue Mar 25, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Mar 25, 2026
@iancha1992 iancha1992 added this pull request to the merge queue Mar 25, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Mar 25, 2026
@tyler-french
Copy link
Copy Markdown
Contributor Author

tyler-french commented Mar 25, 2026

@iancha1992 Let me see if this failure is legit:

//src/test/py/bazel:external_repo_completion_test                        FAILED in 3 out of 3 in 0.6s

EDIT: it seems completely unrelated, OSError: [Errno 5] Input/output error seems transient

@iancha1992 iancha1992 added this pull request to the merge queue Mar 26, 2026
Merged via the queue into bazelbuild:release-8.7.0 with commit d690f14 Mar 26, 2026
46 checks passed
@github-actions github-actions Bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Mar 26, 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