Skip to content

Conversation

@tseaver
Copy link
Contributor

@tseaver tseaver commented May 7, 2021

Also, add retry support to ACL.save, ACL.save_predefined, and ACL.clear.

Toward #38.

Note: this PR is based on 44eeeb4 from PR #431. I will rebase to master when that PR lands. Done.

@tseaver tseaver requested review from a team, andrewsg and crwilcox May 7, 2021 19:36
@tseaver tseaver requested a review from a team as a code owner May 7, 2021 19:36
@product-auto-label product-auto-label bot added the api: storage Issues related to the googleapis/python-storage API. label May 7, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label May 7, 2021
@tseaver tseaver force-pushed the 38-refactor-client-_patch_resource branch 2 times, most recently from 4cdf0c4 to 5dd03f6 Compare May 10, 2021 15:24
@tseaver tseaver force-pushed the 38-refactor-client-_patch_resource branch from 5dd03f6 to aa6d3a0 Compare May 10, 2021 16:13
@tseaver tseaver marked this pull request as draft May 10, 2021 18:38
tseaver added 11 commits June 7, 2021 10:33
Also, add retry support to 'ACL.save', 'ACL.save_predefined', and 'ACL.clear'.

Toward #38.
Indirectly, via 'ACL.save'.

Note:  this change also forwards the retry policy through to the 'ACL.save'
call.

Toward #38.
Indirectly, via 'ACL.save'.

Note:  this change also forwards the retry policy through to the 'ACL.save'
call.

Toward #38.
Indirectly, via 'ACL.save'.

Note:  this change also forwards the retry policy through to the 'ACL.save'
call.

Toward #38.
Indirecly, via 'ACL.save'.

Also, adds 'timeout' and 'retry' support to 'Blob.make_public'.

Toward #38.
Indirecly, via 'ACL.save'.

Also, adds 'timeout' and 'retry' support to 'Blob.make_private'.

Toward #38.
@tseaver tseaver force-pushed the 38-refactor-client-_patch_resource branch from efd5a14 to 9e030cb Compare June 7, 2021 14:33
@tseaver tseaver marked this pull request as ready for review June 7, 2021 14:34
Copy link

@chrisrossi chrisrossi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@tseaver tseaver merged commit e96f852 into master Jun 7, 2021
@tseaver tseaver deleted the 38-refactor-client-_patch_resource branch June 7, 2021 16:01
cojenco pushed a commit to cojenco/python-storage that referenced this pull request Oct 13, 2021
Add retry support to 'ACL.save', 'ACL.save_predefined', and 'ACL.clear'.

Add 'timeout' and 'retry' support to 'Blob.make_private'.

Toward googleapis#38.
cojenco pushed a commit to cojenco/python-storage that referenced this pull request Oct 13, 2021
Add retry support to 'ACL.save', 'ACL.save_predefined', and 'ACL.clear'.

Add 'timeout' and 'retry' support to 'Blob.make_private'.

Toward googleapis#38.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: storage Issues related to the googleapis/python-storage API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants