-
Notifications
You must be signed in to change notification settings - Fork 279
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
added a 'report missing' feature #103
base: master
Are you sure you want to change the base?
Conversation
Would like to use this feature, merge please! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would be quite nice to have this feature
Expecting this feature |
it'll be great with this feature, merge please ! in advance thanks |
I modified it on this basis and changed it to report missing if there is report_missing in the configuration file, because some of my services need report missing, and some don’t
|
@ncabatoff Any updates on this? |
now ? |
Very useful feature for us, we track about a half a dozen system level processes and if they fail to start on boot then we never whether they started or not. Can we get this merged @ncabatoff ? |
@wardellc , it looks like there are merge conflicts now. Mind fixing those and maybe @ncabatoff can merge this? I got caught by this after a reboot just like everyone else.. 😞 |
Very expected feature =( |
Would love this feature! |
Added a "report.missing" command line option. When enabled this will report processes which are not running when process-exporter is first ran as having a num_procs of zero. E.g. if firefox isn't running when process-exporter is started and "firefox" is in a "comm" section, the following metric will be reported:
It uses the "name" field from the process-exporter config file and if the "name" field does not exist it uses all of the process names from the "comm" or "exe" arrays.
This provides a solution to this issue: #59