Skip to content

EDU-4301: Add europe-west3, us-east4 psc service attachment ids#3500

Merged
fairlydurable merged 3 commits intomainfrom
cludden/psc-endpoints
Apr 7, 2025
Merged

EDU-4301: Add europe-west3, us-east4 psc service attachment ids#3500
fairlydurable merged 3 commits intomainfrom
cludden/psc-endpoints

Conversation

@cludden
Copy link
Member

@cludden cludden commented Apr 1, 2025

What does this PR do?

Add europe-west3, us-east4 psc service attachment ids

Notes to reviewers

@cludden cludden requested a review from a team as a code owner April 1, 2025 17:33
@CLAassistant
Copy link

CLAassistant commented Apr 1, 2025

CLA assistant check
All committers have signed the CLA.

@fairlydurable
Copy link
Contributor

@cludden Does gcpregions.md need updating too? Could it be done in this PR if yes? Also, is this information used anywhere else?

File: docs/production-deployment/cloud/references/regions/gcpregions.md

Thanks

@fairlydurable fairlydurable changed the title Add europe-west3, us-east4 psc service attachment ids EDU-4301: Add europe-west3, us-east4 psc service attachment ids Apr 2, 2025
@fairlydurable fairlydurable added the cross-team This issue or PR was submitted from within Temporal label Apr 2, 2025
@cludden
Copy link
Member Author

cludden commented Apr 3, 2025

These aren't new regions, just including the relevant private service connect details. Unsure if it's used anywhere else.

@fairlydurable
Copy link
Contributor

Thank you @cludden!

Copy link
Contributor

@fairlydurable fairlydurable left a comment

Choose a reason for hiding this comment

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

Looks good.

@fairlydurable fairlydurable merged commit 54b8a1a into main Apr 7, 2025
4 checks passed
@fairlydurable fairlydurable deleted the cludden/psc-endpoints branch April 7, 2025 14:38
Comment on lines +61 to +66
| Region | Private Service Connect Service Name |
| -------------- | ------------------------------------------------------------------------------------------ |
| `asia-south1` | `projects/prod-d5spc2sfeshws33bg33vwdef7/regions/asia-south1/serviceAttachments/pl-7w7tw` |
| `europe-west3` | `projects/prod-kwy7d4faxp6qgrgd9x94du36g/regions/europe-west3/serviceAttachments/pl-acgsh` |
| `us-east4` | `projects/prod-y399cvr9c2b43es2w3q3e4gvw/regions/us-east4/serviceAttachments/pl-8awsy` |
| `us-west1 ` | `projects/prod-rbe76zxxzydz4cbdz2xt5b59q/regions/us-west1/serviceAttachments/pl-94w0x` |
Copy link
Contributor

Choose a reason for hiding this comment

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

Asking just based on superficially seeing regions and "private service connect" in both places, but should https://github.com/temporalio/documentation/blob/main/docs/production-deployment/cloud/references/regions/private-service.md?plain=1 also contain the regions listed here?

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

Labels

cross-team This issue or PR was submitted from within Temporal

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants