-
-
Notifications
You must be signed in to change notification settings - Fork 167
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
[FEAT] Add OpenMetrics / Prometheus metrics endpoint #74
Comments
Hey, so the idea would be to allow Prometheus to query smart data from Scrutiny? |
There seems to a not particularly maintained exporter here (last interaction of the maintainer was over a year ago): https://github.com/Sheridan/smartctl_exporter The only other option I have found is to do something like this with a node_exporter script, but this example script leaves much interpretation to be desired (as noted in the top comment): https://github.com/prometheus-community/node-exporter-textfile-collector-scripts/blob/master/smartmon.sh |
Hey! Is there any work on this feature? Or is this abandoned? |
Hey, I've been looking for the same thing! I've been using Scrutiny for a very long time and would like to bank on its reliability to push stats in Prometheus and Grafana. |
I'm open to contributions to support this kind of functionality, however I dont use Prometheus so I will be unable to develop this myself unfortunately |
Bringing this feature will benefit Windows users as well, as there are no exporters for that on Windows (that I know of). |
Whilst I don't have the skills to develop this, I'm more than happy to test it if it gets developed. |
I think this could be realized in running PCP's pmcd, pmda-smart (to read smart data) and pmproxy which can offer the data in Openmetrics format. |
It would be very nice to be able to pull data from scrutiny into Prometheus. This would allow me to leverage my standardized alert system.
https://prometheus.io/docs/instrumenting/writing_exporters/
It should feasibly be a
/metrics
route that translates the results of a DB query into the simple OpenMetrics format. If there is interest in merging this feature, I can look into getting together a PR.The text was updated successfully, but these errors were encountered: