Skip to content
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

ITL: Add --exclude-process to check_procs #10232

Merged
merged 1 commit into from
Nov 15, 2024

Conversation

oxzi
Copy link
Member

@oxzi oxzi commented Nov 13, 2024

For check_procs, both the Monitoring Plugins' implementation1 and the Nagios Plugin2 are supporting the "-X" or "--exclude-process" flag to exclude one or many processes by name. However, this flag is missing here in the Icinga Template Library.

The Nagios Plugin implementation also comes with "-j" and "-g" for FreeBSD jails and Linux cgroups, respectively. But, to keep it compatible, I would ignore these for the moment.

Closes #10226.

Footnotes

  1. https://www.monitoring-plugins.org/doc/man/check_procs.html

  2. https://nagios-plugins.org/doc/man/check_procs.html

For check_procs, both the Monitoring Plugins' implementation[0] and the
Nagios Plugin[1] are supporting the "-X" or "--exclude-process" flag to
exclude one or many processes by name. However, this flag is missing
here in the Icinga Template Library.

The Nagios Plugin implementation also comes with "-j" and "-g" for
FreeBSD jails and Linux cgroups, respectively. But, to keep it
compatible, I would ignore these for the moment.

Closes #10226.

[0]: https://www.monitoring-plugins.org/doc/man/check_procs.html
[1]: https://nagios-plugins.org/doc/man/check_procs.html
@oxzi oxzi added the area/itl Template Library CheckCommands label Nov 13, 2024
@cla-bot cla-bot bot added the cla/signed label Nov 13, 2024
Copy link

@bittorf bittorf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this fixes it for me, i can now use the option

-X = "process-exclude-pattern"

@yhabteab yhabteab added the enhancement New feature or request label Nov 15, 2024
@yhabteab yhabteab added this to the 2.15.0 milestone Nov 15, 2024
@yhabteab yhabteab merged commit f3e7f19 into master Nov 15, 2024
27 checks passed
@yhabteab yhabteab deleted the itl-check_procs-exclude-process branch November 15, 2024 09:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/itl Template Library CheckCommands cla/signed enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

support for check_procs argument --exclude-process
4 participants