Skip to content

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

Merged
yhabteab merged 1 commit intomasterfrom
itl-check_procs-exclude-process
Nov 15, 2024
Merged

ITL: Add --exclude-process to check_procs#10232
yhabteab merged 1 commit intomasterfrom
itl-check_procs-exclude-process

Conversation

@oxzi
Copy link
Copy Markdown
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
Copy Markdown

@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
@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