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

RepositoryCache: add blake3 key type #21998

Closed

Conversation

sluongng
Copy link
Contributor

Support storing repository cache with blake3 hash function.

@github-actions github-actions bot added team-ExternalDeps External dependency handling, remote repositiories, WORKSPACE file. awaiting-review PR is awaiting review from an assigned reviewer labels Apr 15, 2024
@sluongng sluongng force-pushed the sluongng/repo-cache-blake3 branch 3 times, most recently from 2d547d0 to 67ff8d7 Compare April 15, 2024 10:20
@sgowroji sgowroji added awaiting-user-response Awaiting a response from the author and removed awaiting-review PR is awaiting review from an assigned reviewer labels Apr 16, 2024
@sgowroji
Copy link
Member

@sluongng Could you please take a look at the failing checks?

@sluongng sluongng force-pushed the sluongng/repo-cache-blake3 branch 3 times, most recently from d68c18b to 71e695e Compare April 16, 2024 08:53
@sluongng
Copy link
Contributor Author

@sgowroji I think there has been some flakiness in the Bazel CI setup. I have checked the old failures as well as the most recent ones and I don't think they are related to my change.

@sluongng sluongng requested a review from fmeum April 16, 2024 09:26
@sgowroji sgowroji added awaiting-review PR is awaiting review from an assigned reviewer and removed awaiting-user-response Awaiting a response from the author labels Apr 16, 2024
@sluongng sluongng force-pushed the sluongng/repo-cache-blake3 branch from 71e695e to 3a2340a Compare April 16, 2024 09:33
Support storing repository cache with blake3 hash function.
@sluongng sluongng force-pushed the sluongng/repo-cache-blake3 branch from 3a2340a to 1208d34 Compare April 16, 2024 09:35
@fmeum
Copy link
Collaborator

fmeum commented Apr 16, 2024

@meteorcloudy Could you review this?

@meteorcloudy meteorcloudy added awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally and removed awaiting-review PR is awaiting review from an assigned reviewer labels Apr 16, 2024
@fmeum
Copy link
Collaborator

fmeum commented Apr 16, 2024

@bazel-io fork 7.2.0

bazel-io pushed a commit to bazel-io/bazel that referenced this pull request Apr 17, 2024
Support storing repository cache with blake3 hash function.

Closes bazelbuild#21998.

PiperOrigin-RevId: 625727463
Change-Id: I2198adac617c5d6301e56694fe416f3130c61033
github-merge-queue bot pushed a commit that referenced this pull request Apr 18, 2024
Support storing repository cache with blake3 hash function.

Closes #21998.

PiperOrigin-RevId: 625727463
Change-Id: I2198adac617c5d6301e56694fe416f3130c61033

Commit
dc5ec2b

Co-authored-by: Son Luong Ngoc <[email protected]>
@sgowroji sgowroji removed the awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally label Apr 26, 2024
Kila2 pushed a commit to Kila2/bazel that referenced this pull request May 13, 2024
Support storing repository cache with blake3 hash function.

Closes bazelbuild#21998.

PiperOrigin-RevId: 625727463
Change-Id: I2198adac617c5d6301e56694fe416f3130c61033
@iancha1992
Copy link
Member

The changes in this PR have been included in Bazel 7.2.0 RC1. Please test out the release candidate and report any issues as soon as possible.
If you're using Bazelisk, you can point to the latest RC by setting USE_BAZEL_VERSION=7.2.0rc1. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-ExternalDeps External dependency handling, remote repositiories, WORKSPACE file.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants