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

Add option to use short names instead of FQDN #421

Merged
merged 1 commit into from
Feb 3, 2022

Conversation

Jarema
Copy link
Member

@Jarema Jarema commented Feb 3, 2022

Currently if cluster has custom domain name, users have to override k8sClusterDomain: cluster.local.
This is not very portable if you want to reuse the same config for different clusters.

Added optional (disabled by default) FQDN usage.
If

useFQDN: false

is set, NATS will use short DNS names (i.e. nats-0.nats.default)

EDIT: we could consider to use this option by default, though I didn't want to introduce such a change without knowing context.

@Jarema Jarema force-pushed the jarema/add-optional-fdqdn branch from 154ac5f to 23918e6 Compare February 3, 2022 12:29
@Jarema Jarema requested review from variadico and wallyqs February 3, 2022 12:36
Copy link
Member

@derekcollison derekcollison left a comment

Choose a reason for hiding this comment

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

LGTM

@Jarema Jarema merged commit 0bd809f into main Feb 3, 2022
@lucj
Copy link

lucj commented Feb 3, 2022

What is the next release this PR will be part of ?

@variadico variadico deleted the jarema/add-optional-fdqdn branch February 3, 2022 21:03
@derekcollison
Copy link
Member

We will get this out next week.

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