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/document reliable way to figure out if server node joined cluster successfully #880

discordianfish opened this issue Apr 22, 2015 · 2 comments


Copy link


I'm trying to create a AWS AutoScaling configuration to fully automate rolling upgrades of my consul nodes. The idea is that the ASG will terminate one instance, create a new one and wait with the next one until I signal success for the first one.
Now I need to know when a consul server node successfully joined the cluster (-> has all state and is part of the cluster), aka when it's save to kill another server node, so I can signal success.

I might be able to check if the node is in /v1/status/peers but I'm not sure this goes through the raft log and is guaranteed to be consistent. Or I assume that a node sucesffully joined as soon as /v1/status/leader returns something, but same here - I'm not sure about the consistency guarantees.

So if there is a way to programatically tell that a node successfully joined it should be documented. If not, I think it would be useful to have a API endpoint for that.

Copy link

armon commented Apr 23, 2015

The /v1/status/peers does require a Raft log to be updated, so if that is updated, then you should be good to go!

Copy link
Contributor Author

@armon Great, thanks! I think this should be documented? I'd say this is a concern everyone who is replacing nodes in the cluster automatically has. But I'm good, so feel free to close this ticket if you want.

duckhan pushed a commit to duckhan/consul that referenced this issue Oct 24, 2021
Use /tmp/scripts folder instead of the WORKDIR which is / to avoid getting permission denied errors when security context defaults to the non-root user
duckhan pushed a commit to duckhan/consul that referenced this issue Oct 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
None yet

No branches or pull requests

2 participants