-
Notifications
You must be signed in to change notification settings - Fork 23
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
Additional Logstash Metrics #87
Comments
hello @excalq sorry for long time to respond, but i was so tired i wanted to hold looking on the issue
for elasticsearch output plugin i think you can implement that, however for version 2.x i would like to implement a way to scrape custom plugin metrics, just not sure about that. but for 1.x it should be really useful and apperciated! I haven't checked each of the metrics that you provided, but they all seem reasonable to add. If it is not a problem, please split them into 3 separate PRs (this will make PR review process easier):
Thank you for contributions to this project! |
Thanks! Agree on the pre-calculated stats, if they're aggregating another metric... However, the load average really shouldn't be considered a "CPU" load as much as a "system load", as it's the metric of processes using or waiting on the CPU. https://scoutapm.com/blog/understanding-load-averages is a good article on it as well. Situations like high I/O waits will manifest as low CPU utilization but very high system load averages - a common failure mode for data pipeline systems. (Though likely more common in Elasticsearch than Logstash). I can make commits into a tranche of small PRs, which helps me get them out while doing all the other things! And for v2.x custom metrics, especially for plugins is a great idea. For these, I'm only planning to gather common metrics to all pipeline inputs/filters/outputs, with the exception of the Elasticsearch/OpenSearch output plugins. |
I'd like to propose the addition of some additional metrics, for use with a Grafana dashboard. (Derived from an existing one on grafana.com's dashboard registry.)
I'll be happy to contribute a PR with the following metrics added:
Standard via "stock Logstash"
/_node/stats
API response.From the Elasticsearch output plugin (via
/_node/stats
API response)Would these (or a subset) be welcome additions to the set that's currently exported?
UPDATE: Checking these off as they are implemented in other PRs.
The text was updated successfully, but these errors were encountered: