feat: /proc/mdstat collection#9101
feat: /proc/mdstat collection#9101sspaink merged 57 commits intoinfluxdata:masterfrom johnseekins:master
Conversation
|
Not really sure how to fix this lint complaint: There are other input plugins that also violate this limit (e.g. |
|
Sorry about all the commits, but |
Looks like new artifacts were built from this PR. Get them here!Artifact URLs |
Looks like new artifacts were built from this PR. Get them here!Artifact URLs |
Looks like new artifacts were built from this PR. Get them here!Artifact URLs |
ssoroka
left a comment
There was a problem hiding this comment.
Looks great. I have some comments below
|
Thanks for the new plugin! As a user of Linux mdraid arrays, I'm keen to monitor them! I ran it on a system here with four disks, one which is failed, and I don't see it detecting the failed disk? Perhaps the disk just dropped but hasn't failed? Here's the Here's the output: |
…nd failed disk cases
…nd failed disk cases
This should be resolved with the most recent commits. A disk marked as |
Looks like new artifacts were built from this PR. Get them here!Artifact URLs |
|
The same collector was approved and merged into Prometheus' node_exporter: prometheus/procfs#380 |
|
@ssoroka I believe I've addressed all your concerns. Any chance you could take another look at this? |
Required for all PRs:
This adds a new input that collects stats on Linux MD arrays on a system. It is based off https://github.com/prometheus/procfs/blob/master/mdstat.go. (which is apache licensed vs. telegraf's MIT.