-
Notifications
You must be signed in to change notification settings - Fork 1.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Re-enable 'Bucket.requester_pays' feature #4056
Conversation
@tseaver How much of this has been reviewed and how much is new? (Sorry if that is a question I should know the answer to.) |
@dhermes This was all reviewed back in June: I just re-built the branch because the reversion made it hard to see what would be landed this time. |
@tseaver Great. So no review needed? |
@dhermes Not until we resolve whether / how we will implement the additional system tests. |
d34dfe5
to
ec34758
Compare
@lukesneeringer I believe that this branch is ready to merge to
|
Unrelated AppVeyor failure (see #4128). |
@lukesneeringer AFAIK, this branch is ready to merge. |
ec34758
to
41fca6f
Compare
Also, add 'requester_pays' argument to 'Client.create_bucket'. Add a system test which exercises the feature. 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. Toward #3474.
* Add abstract '_PropertyMixin.user_project' property. * Support 'user_project' in '_PropertyMixin.{reload,patch}'. * Add 'user_project' param to 'Bucket.__init__'. * Save and expose via read-only 'user_project' property. * Implement 'Blob.user_property' via bucket's value.
* Block 'Bucket.create' if 'user_project' set: the API does not accept that parameter.
Back-end used to reject it, but now allows it.
…ct' set (#4084) * Pass through extra posargs for system tests. * Plumb 'user_project' arg through 'Client.bucket'.
Test 'Blob.rewrite' w/ 'user_project' set.
41fca6f
to
b0e1c96
Compare
Closes #3474.
Needs extra system tests which pass
user_project
to the various APIs which take it, in the context of a bucket which is created withrequester_pays
enabled.