Skip to content

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

Merged
ycombinator merged 1 commit intoelastic:6.xfrom
ycombinator:6-x/x-pack/monitoring/ccr/monitoring-indices
Sep 18, 2018
Merged

[CCR Monitoring] Only collect stats for specified indices#33655
ycombinator merged 1 commit intoelastic:6.xfrom
ycombinator:6-x/x-pack/monitoring/ccr/monitoring-indices

Conversation

@ycombinator
Copy link
Copy Markdown
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
Copy Markdown
Collaborator

Pinging @elastic/es-core-infra

@jasontedor jasontedor added :Distributed/CCR Issues around the Cross Cluster State Replication features and removed :Core/Infra/Monitoring DEPRECATED, DO NOT USE labels Sep 13, 2018
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-distributed

Copy link
Copy Markdown
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/CCR Issues around the Cross Cluster State Replication features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants