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 example format for initial-cluster in configuration file #15938

Merged
merged 1 commit into from
May 23, 2023

Conversation

jmhbnz
Copy link
Member

@jmhbnz jmhbnz commented May 23, 2023

In discussion #15837 an etcd user wasn't clear about how to specify initial-cluster in the configuration file which led to their cluster not starting, and unfortunately our sample configuration file currently doesn't provide any real guidance on this.

Let's help our users get it right the first time by adding a quick example format to the config file.

Longer term we should also investigate better handling of spaces in these strings so a more expressive dictionary style could be freely used as well.

etcd.conf.yml.sample Outdated Show resolved Hide resolved
@jmhbnz jmhbnz force-pushed the clarify-initial-cluster branch 2 times, most recently from 68c041e to 1446753 Compare May 23, 2023 08:11
Copy link
Member

@ahrtr ahrtr left a comment

Choose a reason for hiding this comment

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

LGTM

Thanks @jmhbnz

Signed-off-by: James Blair <mail@jamesblair.net>
@ahrtr ahrtr merged commit a4f8f56 into etcd-io:main May 23, 2023
@jmhbnz jmhbnz deleted the clarify-initial-cluster branch July 27, 2023 02:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants