Skip to content
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

feat: support custom universe domains/TPC #1212

Merged
merged 13 commits into from
Feb 6, 2024
Merged

feat: support custom universe domains/TPC #1212

merged 13 commits into from
Feb 6, 2024

Conversation

andrewsg
Copy link
Contributor

No description provided.

@andrewsg andrewsg requested review from a team as code owners January 18, 2024 23:57
@product-auto-label product-auto-label bot added size: l Pull request size is large. api: storage Issues related to the googleapis/python-storage API. labels Jan 18, 2024
@andrewsg andrewsg added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jan 19, 2024
@andrewsg andrewsg changed the title feat: allow custom universe domains feat: support custom universe domains/TPC Jan 19, 2024
Copy link
Contributor

@JesseLovelace JesseLovelace left a comment

Choose a reason for hiding this comment

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

Thanks Andrew!

Copy link
Contributor

@cojenco cojenco left a comment

Choose a reason for hiding this comment

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

Looks great, thanks Andrew! Just a few minor questions

google/cloud/storage/client.py Show resolved Hide resolved
google/cloud/storage/_http.py Outdated Show resolved Hide resolved
google/cloud/storage/blob.py Show resolved Hide resolved
Copy link

@BrennaEpp BrennaEpp left a comment

Choose a reason for hiding this comment

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

LGTM - thank you Andrew!

Please pull in googleapis/conformance-tests#89 and make required changes to your testing code to test the new cases (this can be in a separate PR).

google/cloud/storage/blob.py Outdated Show resolved Hide resolved
@andrewsg andrewsg removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Feb 6, 2024
@andrewsg andrewsg merged commit f4cf041 into main Feb 6, 2024
16 checks passed
@andrewsg andrewsg deleted the tpc-signed-url branch February 6, 2024 21:22
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 googleapis/python-storage API. size: l Pull request size is large.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants