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

Expose more metrics about segments, search type, ... #115

Closed
gehel opened this issue Dec 19, 2017 · 8 comments
Closed

Expose more metrics about segments, search type, ... #115

gehel opened this issue Dec 19, 2017 · 8 comments
Milestone

Comments

@gehel
Copy link

gehel commented Dec 19, 2017

Thanks for the great exporter! We are migrating from graphite to prometheus and using this exporter to expose elasticsearch metrics. There are a few metrics we used to collect that don't seem to be exposed:

  • elasticsearch.indices.segments.terms_memory_in_bytes
  • elasticsearch.indices.segments.index_writer_memory_in_bytes
  • elasticsearch.indices.completion.size_in_bytes
  • elasticsearch.indices.segments.norms_memory_in_bytes
  • elasticsearch.indices.segments.stored_fields_memory_in_bytes
  • elasticsearch.indices.segments.doc_values_memory_in_bytes
  • elasticsearch.indices.segments.fixed_bit_set_memory_in_bytes
  • elasticsearch.indices.search.groups.prefix.query_total
  • elasticsearch.indices.search.groups.prefix.query_time_in_millis
  • elasticsearch.indices.suggest.total
  • elasticsearch.indices.suggest.time_in_millis
  • elasticsearch.indices.search.groups.full_text.query_total
  • elasticsearch.indices.search.groups.full_text.query_time_in_millis
  • elasticsearch.indices.search.groups.more_like.query_total
  • elasticsearch.indices.search.groups.more_like.query_time_in_millis

All those metrics are available through the node stat API. Our current implementation based on diamond and is available here.

Any chance you could add those as well?

Thanks!

@gehel
Copy link
Author

gehel commented Dec 19, 2017

For reference, this is tracked on our side at https://phabricator.wikimedia.org/T181627

@dominikschulz
Copy link
Contributor

At first sight it looks reasonable to add those metrics but I want to check back with @zwopir

@zwopir
Copy link
Member

zwopir commented Dec 19, 2017

Yes, looks reasonable. If all supported ES versions expose the metrics, it's just a matter of adding a few struct fields - no fancy logic involved. I'll check it asap. If not we need to find a solution how to handle different json input from the stats endpoint.

@gehel
Copy link
Author

gehel commented Dec 19, 2017

Looking at the history of our diamond collector, we changed almost nothing since 2 years ago. I would need to check what version of elasticsearch we were running at that time, but it should be compatible back to at least 2.x, possibly 1.8. We are running this collector on 5.5 at the moment. I have not checked compatibility with ES 6.x yet.

Thanks for looking into this!

@dominikschulz dominikschulz added this to the 1.0.2 milestone Dec 20, 2017
dominikschulz pushed a commit to dominikschulz/elasticsearch_exporter that referenced this issue Jan 9, 2018
dominikschulz added a commit that referenced this issue Jan 9, 2018
@gehel
Copy link
Author

gehel commented Jan 9, 2018

Thanks! I'll test this as soon as I find some time...

@dominikschulz
Copy link
Contributor

@gehel Thanks! I know that problem wrt. time ;) I'm sorry this took so long, somehow my PR got lost over Christmas.

@gehel
Copy link
Author

gehel commented Feb 6, 2018

After far too long, I finally deployed that new version. All looks good! Thanks for the help!

@dominikschulz
Copy link
Contributor

@gehel Amazing. Thanks for reporting back!

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

No branches or pull requests

3 participants