Re: check_by_ssh: added option to exit with an warning, if there is output on STDERR (#1301)
jklippel <[email protected]>
| Newsgroups | gmane.network.nagios.devel |
|---|---|
| Message-ID | <monitoring-plugins/monitoring-plugins/pull/1301/[email protected]> |
While it is a good idea to centralize matching functionality I am opposed to replacing this small patch by an negate-enhancement. - The size of this patch (i12:d2) alone does not justify the effort to enhance negate by such a big feature.. - There is already the --skip-stderr parameter, the proposed patch only extends the already existing behaviour by issuing a warning. - Using negate would introduce at least one additional fork which is not helpful in a big environment using check_by_ssh very often and a lot of overhead for so little functionality. -- Reply to this email on GitHub: https://github.com/monitoring-plugins/monitoring-plugins/pull/1301#issuecomment-66908797