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

Enable automatic leader election by default for new installs #16224

Open
drewmiranda-gl opened this issue Aug 21, 2023 · 3 comments
Open

Enable automatic leader election by default for new installs #16224

drewmiranda-gl opened this issue Aug 21, 2023 · 3 comments

Comments

@drewmiranda-gl
Copy link
Member

What?

Builds off of the work from #11373 .

Enable automatic leader election by default for new installs

leader_election_mode=automatic

Currently in https://github.com/Graylog2/graylog2-server/blob/master/misc/graylog.conf we have is_leader = true.

It may also be useful to allow nodes participating in leader_election_mode=automatic to not be eligible (although it is possible that omitting leader_election_mode and setting is_leader = false accomplishes this.

Why?

Currently, and in line with current documentation, the leader role is explicitly configured for graylog clusters. This creates a single point of failure for graylog clusters. Allowing automatic leader election will simplify the graylog cluster creation process and can allow clusters to, by default, have better resiliency if the master node is offline or has a failure.

@thll
Copy link
Contributor

thll commented Aug 22, 2023

Currently, leader_election_mode=automatic only works correctly when also run_migrations=false is configured. Migrations then need to be executed manually by running the graylog migrate command. This could be documented but is not very convenient and we should strive to find a better solution for this before considering to make automatic leader election the default.

Also see #16234

@drewmiranda-gl
Copy link
Member Author

Understand. More was just recently shared in slack about this so me opening this may have been premature :(

@thll
Copy link
Contributor

thll commented Aug 22, 2023

Understand. More was just recently shared in slack about this so me opening this may have been premature :(

To the contrary. I think it's good to get the conversation going!

@bernd bernd added the triaged label Aug 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants