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

Temporal Nexus: Self Hosted and Enable Nexus #3140

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

jsundai
Copy link
Contributor

@jsundai jsundai commented Oct 9, 2024

What does this PR do?

Preview

Notes to reviewers


Nexus is only supported in single cluster setups for the time being due to endpoint registry replication not being implemented yet.
Enable Nexus in your self-hosted Temporal Service by updating the server's static configuration file and enabling Nexus through dynamic config, then setting the public callback URL and allowed callback addresses.

Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
For a comprehensive guide for operating Nexus workloads in your self-hosted cluster, see https://github.com/temporalio/temporal/blob/main/docs/architecture/nexus.md.


## Enabling Nexus

To enable Nexus in your deployment:
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
To enable Nexus in your deployment:
To enable Nexus in your deployment:
> NOTE: Replace `$PUBLIC_URL` with a URL value that is accessible to external callers or internally within the cluster. At the time of writing external Nexus calls are considered experimental so it should be safe to use the address of an internal load balancer for the frontend service.

Copy link
Member

Choose a reason for hiding this comment

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

I should call this out in the markdown file in the server repo.

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Contributor

@prasek prasek left a comment

Choose a reason for hiding this comment

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

lgtm with @bergundy's suggestions

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.

3 participants