Skip to content

Adding a single retry if a blob digest doesn't match expected#271

Merged
ola-rozenfeld merged 1 commit intobazelbuild:masterfrom
ola-rozenfeld:workaround
Jan 27, 2021
Merged

Adding a single retry if a blob digest doesn't match expected#271
ola-rozenfeld merged 1 commit intobazelbuild:masterfrom
ola-rozenfeld:workaround

Conversation

@ola-rozenfeld
Copy link
Copy Markdown
Contributor

No description provided.

@google-cla google-cla bot added the cla: yes The author signed a CLA label Jan 27, 2021
@ola-rozenfeld
Copy link
Copy Markdown
Contributor Author

Note: this doesn't add any similar checks to BatchReadBlobs (which would be a good idea for another PR, because we are 100% relying on the server to do the right thing).

@ola-rozenfeld ola-rozenfeld requested a review from rubensf January 27, 2021 00:45
@ola-rozenfeld
Copy link
Copy Markdown
Contributor Author

@atetubou : this is the workaround you asked for, PTAL. Thank you!

@atetubou
Copy link
Copy Markdown
Collaborator

Thanks, but update title of PR? We do multiple retry now.

@ola-rozenfeld ola-rozenfeld force-pushed the workaround branch 6 times, most recently from 8178f92 to e321397 Compare January 27, 2021 19:36
pl4nty pushed a commit to pl4nty/remote-apis-sdks that referenced this pull request Apr 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes The author signed a CLA

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants