-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
Fix CCR search bug caused by paused follower indices #64717
Fix CCR search bug caused by paused follower indices #64717
Conversation
…r index name or remote cluster name when paused follower indices existed in the table.
Pinging @elastic/es-ui (Team:Elasticsearch UI) |
@@ -273,7 +273,7 @@ export class FollowerIndicesTable extends PureComponent { | |||
}; | |||
|
|||
const selection = { | |||
onSelectionChange: selectedItems => this.setState({ selectedItems }), | |||
onSelectionChange: newSelectedItems => this.setState({ selectedItems: newSelectedItems }), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had to make this change to satisfy the linter.
@elasticmachine merge upstream |
@elasticmachine merge upstream |
💚 Build SucceededHistory
To update your PR or re-run it, just comment with: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Tested locally and works as expected. 👍
# Conflicts: # x-pack/plugins/cross_cluster_replication/public/app/sections/home/follower_indices_list/components/follower_indices_table/follower_indices_table.js
# Conflicts: # x-pack/plugins/cross_cluster_replication/public/app/sections/home/follower_indices_list/components/follower_indices_table/follower_indices_table.js
Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync. |
Release note
Searching the follower indices table would crash Kibana whenever a follower index was paused. This is because the _ccr/stats API doesn't provide shard information for paused follower indices, but the search logic depended on this information being present. I've changed the logic to remove this dependency, so search behaves as expected.