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

Added etcd hosts failover. #58

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

cloakedcode
Copy link

Allows multiple etcd nodes to be configured and rotates through until it successfully establishes a connection.

@philips
Copy link

philips commented Feb 13, 2015

lgtm

@igor47
Copy link
Collaborator

igor47 commented Feb 19, 2015

hi @cloakedcode ; this PR needs a rebase now that #68 has been merged. also, i would not merge this because it breaks backwards compatibility of existing config files.

can you rework this to add etcd_hosts as a secondary parameter but still also accept the old host and port options? then you can just add what's in host and port to hosts and loop through all of them.

@telmich
Copy link

telmich commented May 13, 2019

Hey! This looks very interesting! Is there any work going on in this issue?

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.

4 participants