Skip to content

Added etcd hosts failover.#58

Open
cloakedcode wants to merge 2 commits intoairbnb:masterfrom
S2Games:etcd_failover
Open

Added etcd hosts failover.#58
cloakedcode wants to merge 2 commits intoairbnb:masterfrom
S2Games:etcd_failover

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