Skip to content

Conversation

@DaveCTurner
Copy link
Contributor

Adds the node's current term and the term and version of the the last-accepted
cluster state to the message reported by the ClusterFormationFailureHelper,
since these values may be of importance when tracking down a cluster formation
failure.

Adds the node's current term and the term and version of the the last-accepted
cluster state to the message reported by the `ClusterFormationFailureHelper`,
since these values may be of importance when tracking down a cluster formation
failure.
@DaveCTurner DaveCTurner added >enhancement v7.0.0 :Distributed Coordination/Cluster Coordination Cluster formation and cluster state publication, including cluster membership and fault detection. labels Jan 15, 2019
@DaveCTurner DaveCTurner requested a review from ywelsch January 15, 2019 12:58
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed

"discovery will continue using %s from hosts providers and %s from last-known cluster state",
resolvedAddresses, clusterStateNodes);
"discovery will continue using %s from hosts providers and %s from last-known cluster state; " +
"node term %d, last-accepted version %d in term %d",
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: perhaps last-accepted *cluster state* version in term

Copy link
Contributor Author

Choose a reason for hiding this comment

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

😐 we just mentioned the cluster state, this message is already very long, and I don't think it will be misinterpreted as a different kind of version as it is. I left it alone.

Copy link
Contributor

Choose a reason for hiding this comment

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

okay

@DaveCTurner DaveCTurner merged commit a2a40c5 into elastic:master Jan 15, 2019
@DaveCTurner DaveCTurner deleted the 2018-01-15-more-details-in-clusterformationfailurehelper branch January 15, 2019 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed Coordination/Cluster Coordination Cluster formation and cluster state publication, including cluster membership and fault detection. >enhancement v7.0.0-beta1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants