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

Guide Update: Adding/Removing Server #5095

Merged
merged 2 commits into from
Dec 17, 2018
Merged

Conversation

kaitlincart
Copy link
Contributor

Added a note for the consul leave / bootstrap_expect interaction in the "Manually Remove Servers" section.

[ci skip]

Copy link
Contributor

@danielehc danielehc left a comment

Choose a reason for hiding this comment

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

Looks good to me, I like the shorter form more than the first one 👍

danielehc added a commit that referenced this pull request Dec 17, 2018
Reusing the same phrasing as  #5095 to provide info on the effects of the `consul leave` command on Consul quorum
@kaitlincart kaitlincart merged commit cb5524e into master Dec 17, 2018
kaitlincart added a commit that referenced this pull request Dec 17, 2018
* first draft of updated content for consul leave

* Changed the consul leave/bootstrap_expect scenario into a note
kaitlincart pushed a commit that referenced this pull request Jan 25, 2019
Reusing the same phrasing as  #5095 to provide info on the effects of the `consul leave` command on Consul quorum
kaitlincart pushed a commit that referenced this pull request Jan 25, 2019
Reusing the same phrasing as  #5095 to provide info on the effects of the `consul leave` command on Consul quorum
@freddygv freddygv deleted the remove-server-update branch June 14, 2019 19:40
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.

2 participants