-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Panic with mountstats at debian wheezy and weird magic with nfs at jessie #571
Comments
Could you attach a file with the raw contents of procfs with the tab characters preserved? I want to compare with the current fixture data we have. Also, what kernel versions are involved here? |
Oh, sorry, I see the versions at the top now. |
@mdlayher That's an issue in the NFS parsing code in procfs. |
Yep. A gist with the full, unmodified output would be great. Probably a different version of the stats that I'm not accounting for. |
@mdlayher Looking at https://github.com/prometheus/procfs/blob/332f6238064950a97bc3ed3f5421f6418537e707/mountstats.go#L511-L552 I bet he has a file using stat version 1.0. You require in this function stat 1.1. |
@mdlayher IMHO this could be fixed by changing line 527 to |
@Savemech, if you could build and test from master to verify, that would be great. |
@SuperQ now works perfectly fine; many thanks guys! |
Host operating system: output of
uname -a
Linux somewheezy 3.2.46 x86_64 GNU/Linux
and
Linux somejessie 3.16.0-4-amd64 #1 SMP Debian 3.16.7-ckt25-2 (2016-04-08) x86_64 GNU/Linux
node_exporter version: output of
node_exporter -version
node_exporter, version 0.14.0 (branch: master, revision: 840ba5d)
build user: root@bb6d0678e7f3
build date: 20170321-12:12:54
go version: go1.7.5
Are you running node_exporter in Docker?
nope
What did you do that produced an error?
^actually there is no
\
symbols, its one big linenfsstat -c
there is actually even more information on /proc/self/mountstats:
^and actually these nfs mountpoints identical, that's just bad me trying remove any sensitive data
What did you expect to see?
actually working node_exporter
What did you see instead?
meanwhile at debian jessie with this:
starts fine, but siliently fails to collect and show any "node_mountstats_nfs_age_seconds_total" data, meanwhile with enabled debug, i did see
and actually nfs collector seems not working too:
at "broken" wheezy vm:
Maybe should i did two separate issues with description wheezy and jessie issues separately ❔
The text was updated successfully, but these errors were encountered: