-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
Add processes status stats in system input plugin #798
Conversation
This add metrics asked here #235 |
this looks OK but I'm a little worried about the processing required for it, this is going to have to read every single process file in /proc/ and parse out all of the information, and then return the processes.... it likely would be more efficient to use |
from what I can tell, |
@sparrc I understand, but I just did what collectd does ( but maybe this is not a right way :D see here: https://github.com/collectd/collectd/blob/master/src/processes.c#L1826 ) |
fair enough....but I don't think this is limited to linux, is it? it looks like gopsutil supports the status info for freebsd, darwin, etc. |
Maybe there is some other status for those OSes ? |
For all unix systems these should be it, I have no idea about Windows, but generally I've been trying to encourage people to use windows perf counters for system metrics |
It returns the number of processus grouped by their status (running/sleeping/zombie/...)