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

Remove join timeout #60873

Merged

Conversation

DaveCTurner
Copy link
Contributor

There is no point in timing out a join attempt any more. Timing out and
retrying with the same master is pointless, and an in-flight join
attempt to one master no longer blocks attempts to join other masters.
This commit removes this unnecessary setting.

Relates #60872 in which this setting was deprecated.

There is no point in timing out a join attempt any more. Timing out and
retrying with the same master is pointless, and an in-flight join
attempt to one master no longer blocks attempts to join other masters.
This commit removes this unnecessary setting.

Relates elastic#60872 in which this setting was deprecated.
@DaveCTurner DaveCTurner added >enhancement >breaking :Distributed Coordination/Cluster Coordination Cluster formation and cluster state publication, including cluster membership and fault detection. v8.0.0 labels Aug 10, 2020
@DaveCTurner DaveCTurner requested a review from ywelsch August 10, 2020 08:34
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed (:Distributed/Cluster Coordination)

@elasticmachine elasticmachine added the Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. label Aug 10, 2020
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 added a commit that referenced this pull request Aug 10, 2020
There is no point in timing out a join attempt any more once a cluster
is entirely in 7.x. Timing out and retrying with the same master is
pointless, and an in-flight join attempt to one master no longer blocks
attempts to join other masters. This commit deprecates this unnecessary
setting and removes its effect from the joining process.

Relates #60873 which removes this setting in master.
@DaveCTurner DaveCTurner merged commit 19eb922 into elastic:master Aug 10, 2020
@DaveCTurner DaveCTurner deleted the 2020-08-10-remove-join-timeout branch August 10, 2020 12:57
masseyke added a commit that referenced this pull request Sep 2, 2021
The cluster.join.timeout setting has been removed in 8.0. This commit adds a deprecation info
check for that setting.
Relates #42404 #60873
jrodewig added a commit that referenced this pull request Sep 16, 2021
We deprecated the `cluster.join.timeout` setting in 7.10 with PR #60872.
However, we didn't add a related item to the 7.10 deprecation docs. This adds
the missing item.

Relates to #60873.
elasticsearchmachine pushed a commit that referenced this pull request Sep 16, 2021
We deprecated the `cluster.join.timeout` setting in 7.10 with PR #60872.
However, we didn't add a related item to the 7.10 deprecation docs. This adds
the missing item.

Relates to #60873.
elasticsearchmachine pushed a commit that referenced this pull request Sep 16, 2021
We deprecated the `cluster.join.timeout` setting in 7.10 with PR #60872.
However, we didn't add a related item to the 7.10 deprecation docs. This adds
the missing item.

Relates to #60873.
elasticsearchmachine pushed a commit that referenced this pull request Sep 16, 2021
We deprecated the `cluster.join.timeout` setting in 7.10 with PR #60872.
However, we didn't add a related item to the 7.10 deprecation docs. This adds
the missing item.

Relates to #60873.
jrodewig added a commit that referenced this pull request Sep 16, 2021
We deprecated the `cluster.join.timeout` setting in 7.10 with PR #60872.
However, we didn't add a related item to the 7.10 deprecation docs. This adds
the missing item.

Relates to #60873.
jrodewig added a commit that referenced this pull request Sep 16, 2021
We deprecated the `cluster.join.timeout` setting in 7.10 with PR #60872.
However, we didn't add a related item to the 7.10 deprecation docs. This adds
the missing item.

Relates to #60873.
jrodewig added a commit that referenced this pull request Sep 16, 2021
We deprecated the `cluster.join.timeout` setting in 7.10 with PR #60872.
However, we didn't add a related item to the 7.10 deprecation docs. This adds
the missing item.

Relates to #60873.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>breaking :Distributed Coordination/Cluster Coordination Cluster formation and cluster state publication, including cluster membership and fault detection. >enhancement Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants