Skip to content

Conversation

@tseaver
Copy link
Contributor

@tseaver tseaver commented Jun 8, 2017

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]
@tseaver tseaver 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
@tseaver tseaver requested review from dhermes and lukesneeringer June 8, 2017 20:11
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 8, 2017
@tseaver
Copy link
Contributor Author

tseaver commented Jun 9, 2017

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

@tseaver tseaver 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
Contributor

dhermes commented Jun 9, 2017

👍

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 Cloud Storage API. cla: yes This 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