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

Expose minimum_master_nodes in cluster state #37811

Merged

Conversation

DaveCTurner
Copy link
Contributor

@DaveCTurner DaveCTurner commented Jan 24, 2019

To safely support rolling upgrades from 6.x to 7.x we need the 7.x nodes to
have access to the minimum_master_nodes setting, but this setting is
otherwise unnecessary in 7.x and we would like to remove it. Since a rolling
upgrade from 6.x to 7.x involves the 7.x nodes joining a 6.x master, we can
avoid the need for setting minimum_master_nodes on the 7.x nodes by copying
the value set on the 6.x master.

This change exposes the master's node-level value for minimum_master_nodes
via a field in the cluster state.

Relates #37701

To safely support rolling upgrades from 6.x to 7.x we need the 7.x nodes to
have access to the `minimum_master_nodes` setting, but this setting is
otherwise unnecessary in 7.x and we would like to remove it.  Since a rolling
upgrade from 6.x to 7.x involves the 7.x nodes joining a 6.x master, we can
avoid the need for setting `minimum_master_nodes` on the 7.x nodes by copying
the value set on the 6.x master.

This change exposes the master's node-level value for `minimum_master_nodes`
via a field in the cluster state.
@DaveCTurner DaveCTurner added >enhancement blocker :Distributed/Cluster Coordination Cluster formation and cluster state publication, including cluster membership and fault detection. v6.7.0 labels Jan 24, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed

@DaveCTurner
Copy link
Contributor Author

The parallel changes to master were in #37701, with the serialisation version constraint set to 7.0.0. We can only merge this while BWC testing is disabled, and BWC testing cannot be enabled until we have adjusted the serialisation version constraint to 6.7.0 in master.

DaveCTurner added a commit to DaveCTurner/elasticsearch that referenced this pull request Jan 24, 2019
This completes the BWC serialisation changes required for a 6.7 master to
inform other nodes of the node-level value of the `minimum_master_nodes`
setting.

Relates elastic#37701, elastic#37811
Copy link
Contributor

@ywelsch ywelsch left a comment

Choose a reason for hiding this comment

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

LGTM

@DaveCTurner
Copy link
Contributor Author

@elasticmachine please run elasticsearch-ci/2

1 similar comment
@DaveCTurner
Copy link
Contributor Author

@elasticmachine please run elasticsearch-ci/2

@DaveCTurner DaveCTurner merged commit 4f08fe0 into elastic:6.x Jan 24, 2019
DaveCTurner added a commit that referenced this pull request Jan 24, 2019
This completes the BWC serialisation changes required for a 6.7 master to
inform other nodes of the node-level value of the `minimum_master_nodes`
setting.

Relates #37701, #37811
@DaveCTurner DaveCTurner deleted the 2019-01-24-expose-minimum-master-nodes branch January 24, 2019 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocker :Distributed/Cluster Coordination Cluster formation and cluster state publication, including cluster membership and fault detection. >enhancement v6.7.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants