Skip to content

Conversation

@lukeknep
Copy link
Contributor

@lukeknep lukeknep commented Dec 2, 2025

What does this PR do?

If a Namespace has API keys and replication enabled, and wants to remove the replica, they must switch to regional endpoints and contact Temporal Support to validate that the switch has been done. This was not called out in the docs before. This PR calls it out and explains why this is required.

Notes to reviewers

@lukeknep lukeknep requested a review from a team as a code owner December 2, 2025 00:18
@CLAassistant
Copy link

CLAassistant commented Dec 2, 2025

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
2 out of 3 committers have signed the CLA.

✅ GSmithApps
✅ lennessyy
❌ lukeknep
You have signed the CLA already but the status is still pending? Let us recheck it.

@vercel
Copy link

vercel bot commented Dec 2, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
temporal-documentation Ready Ready Preview Comment Dec 5, 2025 0:13am

@github-actions
Copy link
Contributor

github-actions bot commented Dec 2, 2025

📖 Docs PR preview links


<TabItem value="webui" label="Web UI">

1. (if using API keys on this Namespace) Ensure that any Workers or Clients with API keys have been configured to use the [regional endpoint](/cloud/regions) for the Namespace's region.
Copy link
Member

Choose a reason for hiding this comment

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

are there directions on how to do this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I will link to the updated section that Grant wrote. Does that sound good?

:::note

After removing a replica, Temporal Cloud can't re-enable replication for a given Namespace for seven days.
Namespaces with API keys enabled need assistance from Temporal Support to remove a replica. Please [contact support](/cloud/support#support-ticket) with the Namespace ID of the Namespace on which to remove a replica. You must confirm that Workers and Clients with API keys have been configured to connect to the Namespace using the published [regional endpoint](/cloud/regions).
Copy link
Member

Choose a reason for hiding this comment

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

we're removing this requirement - is this temporary until this is shipped?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, exactly. It sounds like it will be months until we remove this requirement. In the meantime, we are getting a request like this every week or two.

@GSmithApps
Copy link
Contributor

GSmithApps commented Dec 3, 2025

Plz don't merge until this PR is merged. It piggybacks on this one, and this one is not as complete without it

Edit, that's merged ✅

* added clarifications for API key + high avail

* Update docs/production-deployment/cloud/get-started/namespaces.mdx

Co-authored-by: Lanie Hei <lanie.b.hei@gmail.com>

* Update docs/production-deployment/cloud/get-started/api-keys.mdx

Co-authored-by: Lanie Hei <lanie.b.hei@gmail.com>

* responded to PR comments

---------

Co-authored-by: Lanie Hei <lanie.b.hei@gmail.com>

### With High Availability features

Use the gRPC Namespace endpoint: `<namespace>.<account>.tmprl.cloud:7233`. This allows automated failover without needing to configure worker(s) with both endpoints. These endpoints will still work, but in the event of a failover, there is a brief period of unavailability between your workers and Temporal server.
Copy link
Contributor

@lennessyy lennessyy Dec 4, 2025

Choose a reason for hiding this comment

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

What do you mean by both endpoints? Do you mean the regional endpoints for the original namespace and its replica?

When you say there is a brief period of unavailability, do you mean that users will experience this if they manually configured the two regional endpoints? or will it exist regardless even if they use the <namespace>.<account>.tmprl.cloud:7233 endpoint?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch. I think this rewrite would make more sense:

Use the gRPC Namespace endpoint: <namespace>.<account>.tmprl.cloud:7233. Using the Namespace endpoint is usually preferred in multi-region architectures, because it allows Workers and Clients to always connect to the active region of the Namespace. When a failover is triggered and the Namespace's active region changes, Temporal Cloud uses DNS to point this Namespace endpoint to the new active region.

```

If using API key authentication with the `--api-key` flag, you must add it directly after the tcld command and before `namespace delete-region`
If using the `--api-key` flag to issue this CLI command, you must add it directly after the tcld command and before `namespace delete-region`
Copy link
Contributor

Choose a reason for hiding this comment

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

What does this mean to "Add it directly after the tcld command?" This seems to suggest there is more than 1 command, but we are only showing one command. Do we mean that they must put the --api-key flag in the command?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

There is only one command. Perhaps:

...directly after tcld and before...

lennessyy and others added 2 commits December 3, 2025 16:30
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.

8 participants