check_by_ssh: added option to exit with an warning, if there is output on STDERR (#1301)

jklippel notifications at github.com
Sun Dec 14 11:32:02 CET 2014


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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://www.monitoring-plugins.org/archive/devel/attachments/20141214/9c739e7a/attachment.html>


More information about the Devel mailing list