Skip to content

Support 'user_project' in '_PropertyMixin.{reload,patch}'. - #3489

Closed
tseaver wants to merge 5 commits into
googleapis:masterfrom
tseaver:3474-storage-bucket-requester_pays-_propertymixin-apis
Closed

Support 'user_project' in '_PropertyMixin.{reload,patch}'.#3489
tseaver wants to merge 5 commits into
googleapis:masterfrom
tseaver:3474-storage-bucket-requester_pays-_propertymixin-apis

Conversation

@tseaver

Copy link
Copy Markdown
Contributor

Uses #3488 as a base.

tseaver added 5 commits June 8, 2017 14:53
Toward #3474.
Note that the new system test is skipped, because 'Buckets.insert' fails
with the 'billing/requesterPays' field set, both in our system tests and
in the 'Try It!' form in the docs.
Use it in the (still skipped) system test.
If we can figure out how to determing that the feature is availalbe
programatically later, we can adjust the 'REQUESTER_PAYS_ENABLED' flag,
which will also be available to guard other system tests using it.
[ci skip]
@tseavertseaver added api: storage Issues related to the Cloud Storage API. type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design. labels Jun 8, 2017
@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 8, 2017
@tseaver

Copy link
Copy Markdown
ContributorAuthor

Oops, turns out this one needs to be folded in with #3490.

@tseavertseaver closed this Jun 9, 2017
@tseaver
tseaver deleted the 3474-storage-bucket-requester_pays-_propertymixin-apis branch June 9, 2017 19:19
@dhermes

Copy link
Copy Markdown
Contributor

👍

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: storageIssues related to the Cloud Storage API.cla: yesThis human has signed the Contributor License Agreement.type: feature request‘Nice-to-have’ improvement, new feature or different behavior or design.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@tseaver@dhermes@googlebot