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

[CCR Monitoring] Only collect stats for specified indices #33655

Conversation

ycombinator
Copy link
Contributor

Backport of #33646 to 6.x. Original message:

Follow up to #33617. Relates to #30086.

As with all other per-index Monitoring collectors, the CcrStatsCollector should only collect stats for the indices the user wants to monitor. This list is controlled by the xpack.monitoring.collection.indices setting and defaults to all indices.

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@jasontedor jasontedor added :Distributed Indexing/CCR Issues around the Cross Cluster State Replication features and removed :Data Management/Monitoring labels Sep 13, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed

Copy link
Member

@jasontedor jasontedor left a comment

Choose a reason for hiding this comment

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

LGTM.

)

Follow up to elastic#33617. Relates to elastic#30086.

As with all other per-index Monitoring collectors, the `CcrStatsCollector` should only collect stats for the indices the user wants to monitor. This list is controlled by the `xpack.monitoring.collection.indices` setting and defaults to all indices.
@ycombinator ycombinator force-pushed the 6-x/x-pack/monitoring/ccr/monitoring-indices branch from 6ddbdf5 to 5715044 Compare September 18, 2018 01:58
@ycombinator ycombinator merged commit 80297f4 into elastic:6.x Sep 18, 2018
@ycombinator ycombinator deleted the 6-x/x-pack/monitoring/ccr/monitoring-indices branch September 18, 2018 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Distributed Indexing/CCR Issues around the Cross Cluster State Replication features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants