Skip to content

Conversation

jskeet
Copy link
Collaborator

@jskeet jskeet commented May 30, 2017

It's not as bad as it looks - the vast majority of this code is just repeated for each request.

Completely untested so far. (Need to use two different projects...)
@jskeet jskeet force-pushed the requester-pays branch 2 times, most recently from ae3d48d to 753bf13 Compare May 30, 2017 12:59
Copy link
Contributor

@chrisdunelm chrisdunelm left a comment

Choose a reason for hiding this comment

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

LGTM.
Just one nit.

{
return;
}

This comment was marked as spam.

This comment was marked as spam.

These require two extra environment variables
(REQUESTER_PAYS_TEST_PROJECT and REQUESTER_PAYS_CREDENTIALS) to be
set. If they aren't set, requester-pays testing is not performed.

Note that within the fixture code, we have to specify the
UserProject option for all operations - I don't *think* we should
have to do that, but it looks like it's required right now.
@jskeet jskeet merged commit 65031e5 into googleapis:storage-requester-pays May 30, 2017
@jskeet jskeet deleted the requester-pays branch May 30, 2017 14:55
google-cloud-sdk-librarian-dotnet-robot added a commit that referenced this pull request Aug 8, 2025
feat: Update Compute Engine v1 API to revision 20250729 (#1085)


Source-Link: googleapis/googleapis@a386edb
jskeet pushed a commit that referenced this pull request Aug 8, 2025
feat: Update Compute Engine v1 API to revision 20250729 (#1085)


Source-Link: googleapis/googleapis@a386edb
google-cloud-sdk-librarian-dotnet-robot added a commit that referenced this pull request Aug 8, 2025
### New features

- Update Compute Engine v1 API to revision 20250729 (#1085)
- Update Compute Engine v1 API to revision 20250728 (#1081)

Librarian-Release-Library: Google.Cloud.Compute.V1
Librarian-Release-Version: 3.13.0
Librarian-Release-ID: release-20250808T081232Z
jskeet pushed a commit that referenced this pull request Aug 8, 2025
### New features

- Update Compute Engine v1 API to revision 20250729 (#1085)
- Update Compute Engine v1 API to revision 20250728 (#1081)

Librarian-Release-Library: Google.Cloud.Compute.V1
Librarian-Release-Version: 3.13.0
Librarian-Release-ID: release-20250808T081232Z
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants