-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
docs: Explain what "quorum size" means exactly in "consul leave" docs #11975
Comments
Sorry you found this to be confusing. Thanks for suggesting how we could improve this page.
This information is documented on the Consensus Protocol page, specifically under the Raft Protocol Overview and Raft in Consul sections.
We can look to either link "quorum size" on the page ago a section on the Consensus Protocols, or add a callout briefly stating what quorum size means and linking to the aforementioned page for more info. |
@blake Thanks for the swift reply! Looking at the table https://www.consul.io/docs/architecture/consensus#deployment_table, isn't the statement from the
According to the table, when the Server count is reduced from 7 to 6, the Quorum Size stays at 4, so it's not reduced by 1 in this case. So that would be rather a "can" than "will" (which implies a strict "-1", and added to my confusion whether the table's column is meant or not). If that is correct, I suggest to improve the wording as follows:
|
Ya, I think you are correct. Sometimes we say "quorum size" but really we should say "cluster size". The Quorum size is derived from the cluster size, but it won't always change on a leave, as you point out. |
Much appreciated! |
https://www.consul.io/commands/leave (pinned version) says:
To the operator, it is unclear what "quorum size" means exactly here.
Assuming I ran
consul leave
, how can I check what the new decreased quorum size is?Type
=server
inconsul members
?known_servers
field inconsul info
(which is presumably the same as the above)?Given that the docs warn that the quorum size may drop to 1 and that that is bad, they should say how to check what the quorum size currently is.
The text was updated successfully, but these errors were encountered: