Skip to content

Handle no such remote cluster exception in ccr#53415

Merged
dnhatn merged 3 commits intoelastic:masterfrom
dnhatn:fix-ccr-remote-client
Mar 13, 2020
Merged

Handle no such remote cluster exception in ccr#53415
dnhatn merged 3 commits intoelastic:masterfrom
dnhatn:fix-ccr-remote-client

Conversation

@dnhatn
Copy link
Copy Markdown
Member

@dnhatn dnhatn commented Mar 11, 2020

A remote client can throw a NoSuchRemoteClusterException while fetching the cluster state from the leader cluster. We also need to handle that exception when retrying to add a retention lease to the leader shard.

Closes #53225

@dnhatn dnhatn added >bug :Distributed/CCR Issues around the Cross Cluster State Replication features v8.0.0 v7.7.0 v7.6.2 v6.8.8 labels Mar 11, 2020
@dnhatn dnhatn requested a review from ywelsch March 11, 2020 15:38
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-distributed (:Distributed/CCR)

Copy link
Copy Markdown
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

@dnhatn
Copy link
Copy Markdown
Member Author

dnhatn commented Mar 13, 2020

Thanks Yannick.

@dnhatn dnhatn merged commit 375a211 into elastic:master Mar 13, 2020
@dnhatn dnhatn deleted the fix-ccr-remote-client branch March 13, 2020 15:19
@jakelandis jakelandis added v6.8.9 and removed v6.8.8 labels Mar 19, 2020
dnhatn added a commit that referenced this pull request Apr 4, 2020
A remote client can throw a NoSuchRemoteClusterException while fetching 
the cluster state from the leader cluster. We also need to handle that
exception when retrying to add a retention lease to the leader shard.

Closes #53225
dnhatn added a commit that referenced this pull request Apr 4, 2020
A remote client can throw a NoSuchRemoteClusterException while fetching
the cluster state from the leader cluster. We also need to handle that
exception when retrying to add a retention lease to the leader shard.

Closes #53225
dnhatn added a commit that referenced this pull request Apr 4, 2020
A remote client can throw a NoSuchRemoteClusterException while fetching
the cluster state from the leader cluster. We also need to handle that
exception when retrying to add a retention lease to the leader shard.

Closes #53225
dnhatn added a commit that referenced this pull request Apr 5, 2020
A remote client can throw a NoSuchRemoteClusterException while fetching
the cluster state from the leader cluster. We also need to handle that
exception when retrying to add a retention lease to the leader shard.

Closes #53225
@jakelandis jakelandis removed the v8.0.0 label Jul 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>bug :Distributed/CCR Issues around the Cross Cluster State Replication features v6.8.9 v7.6.2 v7.7.0 v7.8.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

IndexFollowingIT.testUpdateRemoteConfigsDuringFollowing fails on master

5 participants