search

ncabatoff / process-exporter

Prometheus exporter that mines /proc to report on selected processes
MIT License
1.67k stars 265 forks source link

Added remove-empty-groups option #283

Closed zviRosenfeldRedis closed 1 month ago

zviRosenfeldRedis commented 9 months ago

If this option is True (default false) we'll remove groups if they have 0 processes (by default we leave the group, assuming processes from that group will be back)

The only real change is at the end of file proc/grouper.go - where I remove un-found groups if remove-dead-groups is true

fixed https://github.com/ncabatoff/process-exporter/issues/282

ncabatoff commented 4 months ago

This seems like a reasonable option. If you resolve the conflicts I'm open to merging it. Sorry for the long delay!

zviRosenfeldRedis commented 4 months ago

@ncabatoff. Thanks. I've solved the conflicts

ncabatoff commented 1 month ago

Thanks @zviRosenfeldRedis !