-
Notifications
You must be signed in to change notification settings - Fork 796
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
Comments
For reference, this is tracked on our side at https://phabricator.wikimedia.org/T181627 |
At first sight it looks reasonable to add those metrics but I want to check back with @zwopir |
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. |
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! |
Thanks! I'll test this as soon as I find some time... |
@gehel Thanks! I know that problem wrt. time ;) I'm sorry this took so long, somehow my PR got lost over Christmas. |
After far too long, I finally deployed that new version. All looks good! Thanks for the help! |
@gehel Amazing. Thanks for reporting back! |
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:
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!
The text was updated successfully, but these errors were encountered: