Skip to content
This repository was archived by the owner on Mar 31, 2026. It is now read-only.

refactor: add / use 'Client._post_resource' method - #443

Merged
tseaver merged 11 commits into
masterfrom
38-refactor-client-_post_resource
Jun 8, 2021
Merged

refactor: add / use 'Client._post_resource' method#443
tseaver merged 11 commits into
masterfrom
38-refactor-client-_post_resource

Conversation

@tseaver

@tseavertseaver commented May 11, 2021

Copy link
Copy Markdown
Contributor

Toward #38.

Note: this PR is based on 6244df0 of PR #442. I will rebase to master when that PR lands. Done.

@tseaver
tseaver requested review from andrewsg and crwilcoxMay 11, 2021 21:08
@product-auto-labelproduct-auto-labelBot added the api: storage Issues related to the googleapis/python-storage API. label May 11, 2021
@google-clagoogle-claBot added the cla: yes This human has signed the Contributor License Agreement. label May 11, 2021
tseaver added 8 commits June 7, 2021 16:13
Also, add retry support to 'Client.create_hmac_key'.
Toward #38.
Also, add retry support to 'BucketNotfication.create'.
Toward #38.
Also, avoid making API requests in tests for 'Blob.update_storage_class':
mock the blob's 'rewrite' method instead.
Toward #38.
Also, avoid making API requests in tests for 'Bucket.rename_blob':
mock the bucket's 'copy_blob' method instead.
Toward #38.
@tseaver
tseaverforce-pushed the 38-refactor-client-_post_resource branch from 2acfb06 to 24fdae5CompareJune 7, 2021 20:13
@tseaver
tseaver marked this pull request as ready for review June 7, 2021 20:13
@tseaver
tseaver requested review from a teamJune 7, 2021 20:13
@tseavertseaver added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 7, 2021
Comment threadgoogle/cloud/storage/client.py Outdated
Comment threadgoogle/cloud/storage/notification.py Outdated
Comment threadgoogle/cloud/storage/client.py Outdated
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 7, 2021
@chrisrossichrisrossi changed the title refactor: add / use 'Client._post_resorce' methodrefactor: add / use 'Client._post_resource' methodJun 8, 2021

@chrisrossichrisrossi left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM!

@tseaver
tseaver requested a review from andrewsgJune 8, 2021 15:53

@andrewsgandrewsg left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks!

@tseaver
tseaver merged commit ef764ac into masterJun 8, 2021
@tseaver
tseaver deleted the 38-refactor-client-_post_resource branch June 8, 2021 16:49
cojenco pushed a commit to cojenco/python-storage that referenced this pull request Oct 13, 2021
cojenco pushed a commit to cojenco/python-storage that referenced this pull request Oct 13, 2021
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: storageIssues related to the googleapis/python-storage API.cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@tseaver@chrisrossi@andrewsg@yoshi-kokoro