Skip to content

Adds support for collecting elasticsearch index_stats#666

Merged
mx-psi merged 2 commits intoDataDog:masterfrom
charles-ferguson:support-elasticsearch-index-stats
Oct 26, 2020
Merged

Adds support for collecting elasticsearch index_stats#666
mx-psi merged 2 commits intoDataDog:masterfrom
charles-ferguson:support-elasticsearch-index-stats

Conversation

@charles-ferguson
Copy link
Copy Markdown
Contributor

What does this PR do?

Datadog integration now supports collecting index_stats. This PR adds
support to include those stats.

Motivation

We want to collect index stats and we currently install datadog integrations via puppet.

Datadog integration now supports collecting index_stats. This PR adds
support to include those stats.
@charles-ferguson charles-ferguson requested a review from a team as a code owner October 22, 2020 23:56
Copy link
Copy Markdown
Member

@mx-psi mx-psi left a comment

Choose a reason for hiding this comment

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

Looks good to me, just one small thing so that when we merge this CI passes

Comment thread spec/classes/datadog_agent_integrations_elasticsearch_spec.rb Outdated
Co-authored-by: Pablo Baeyens <pbaeyens31+github@gmail.com>
Copy link
Copy Markdown
Member

@mx-psi mx-psi left a comment

Choose a reason for hiding this comment

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

Awesome, thanks for the PR, merging now! 🚀

@mx-psi mx-psi merged commit 92d5db5 into DataDog:master Oct 26, 2020
cegeka-jenkins pushed a commit to cegeka/puppet-datadog_agent that referenced this pull request Feb 5, 2026
* Adds support for collecting elasticsearch index_stats

Datadog integration now supports collecting index_stats. This PR adds
support to include those stats.

* Update spec/classes/datadog_agent_integrations_elasticsearch_spec.rb

Co-authored-by: Pablo Baeyens <pbaeyens31+github@gmail.com>

Co-authored-by: Pablo Baeyens <pbaeyens31+github@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants