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
processes collector is broken after upgrade to 1.2.0 #2110
Comments
I got a same issue too.
I queried
|
Can you try 1.2.1? I think both issues should be fixed there. |
@discordianfish, unfortunately there are still same errors for
|
I can confirm the same error as @karlism using 1.2.1 |
Odd, I don't see any relevant changes between these two releases for that collector. The collector itself hasn't changed at all. And where I'll have to do some more debugging. If people who can reproduce the problem, please include your OS info (uname -a). |
Wrap error returns so that the user can know more about what failed. Returns from getAllocatedThreads() are errors anyway. Related to #2110 Signed-off-by: Ben Kochie <superq@gmail.com>
If someone can try running with Or even better, run the node_exporter with this patch: #2112 |
Got following error with debug log level
|
Thanks, that helps. I'm guessing that's related prometheus/procfs#402. That bugfix didn't make it into the release. |
Update procfs library to include ignored fields ParseInt handling. Wrap error returns so that the user can know more about what failed. Returns from getAllocatedThreads() are errors anyway. Fixes: #2110 Signed-off-by: Ben Kochie <superq@gmail.com>
@jeffrey4l Can you give this patch a try? #2112 |
Update procfs library to include ignored fields ParseInt handling. Wrap error returns so that the user can know more about what failed. Returns from getAllocatedThreads() are errors anyway. Fixes: #2110 Signed-off-by: Ben Kochie <superq@gmail.com>
Update procfs library to include ignored fields ParseInt handling. Wrap error returns so that the user can know more about what failed. Returns from getAllocatedThreads() are errors anyway. Fixes: prometheus#2110 Signed-off-by: Ben Kochie <superq@gmail.com>
Update procfs library to include ignored fields ParseInt handling. Wrap error returns so that the user can know more about what failed. Returns from getAllocatedThreads() are errors anyway. Fixes: prometheus#2110 Signed-off-by: Ben Kochie <superq@gmail.com>
Host operating system: output of
uname -a
$ uname -a
Linux hostname 3.10.0-1160.36.2.el7.x86_64 #1 SMP Wed Jul 21 11:57:15 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
node_exporter version: output of
node_exporter --version
Prior to upgrade:
After an upgrade:
node_exporter command line flags
/usr/bin/node_exporter --no-collector.bonding --no-collector.conntrack --no-collector.fibrechannel --no-collector.infiniband --no-collector.ipvs --no-collector.nfs --no-collector.nfsd --collector.textfile.directory=/var/lib/node_exporter/textfile_collector --collector.ntp --no-collector.pressure --collector.processes --no-collector.rapl --no-collector.zfs --web.listen-address=172.16.xxx.xxx:9100
Are you running node_exporter in Docker?
No
What did you do that produced an error?
Upgrade node exporter from 1.1.2 to 1.2.0
What did you expect to see?
I would expect process collector to work
What did you see instead?
Process exporter is broken
Prior to upgrade:
After an upgrade:
I can also see that metrics like
node_processes_max_processes
are missing after an upgrade.The text was updated successfully, but these errors were encountered: