Add mountstats collector for detailed NFS statistics#367
Add mountstats collector for detailed NFS statistics#367beorn7 merged 1 commit intoprometheus:masterfrom mdlayher:mountstats
Conversation
|
Awesome. @SuperQ, do you want me to tack more changes onto this PR, or make a new one for the additional stats? |
beorn7
left a comment
There was a problem hiding this comment.
Looks good in general, just the confusion about the "items in queue" metrics, if they are Gauges or Counters. (If the latter, the help string should be reworded, something like "total number of items added to the queue" or something.)
collector/mountstats_linux.go
Outdated
There was a problem hiding this comment.
This sounds like it could go down, i.e. it should be a Gauge?
collector/mountstats_linux.go
Outdated
There was a problem hiding this comment.
Same as above, help string suggests a Gauge.
collector/mountstats_linux.go
Outdated
|
@beorn7 good call. According the docs I followed while parsing these stats:
Based on these descriptions, I believe your suggestion of "total number of items added to the queue" will be adequate. |
|
@beorn7 Updated. Please take a look at your convenience. |
|
In lack of more qualified reviewers, I'll approve and merge. |
First pass at a
mountstatscollector for detailed NFS statistics. There are ~25 more statistics I can currently add but I wanted to check in my work before doing a bunch more.Fixes #366 .