invalid perfdata from check_icmp (patch)

[Thomas Guyot-Sionnest]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Andreas Ericsson wrote:
| Wolfgang wrote:
|> Somebody on the german nagios forum used a symbolic link (check_host)
to check_icmp. In hostcheck mode the perfdata output of check_icmp is
somewhat incomplete as no valid warning value is returned which leads to
an error if using functions to process the values.
|>
|
| According to the nagiosplugin guidelines, the warning value isn't required
| in performance data output. A parser expecting to find a value there is
| broken and needs to be fixed. Adding a false warning value to the plugin
| itself is not the right fix.

Although I totally forgot to reply this week-end, that was my thought as
well. While some plugins force users to specify warning and critical
thresholds this behavior (which is broken IMHO) will definitely change
with the new thresholds syntax.

If someone wants some data just to view it from the host detail page,
and/or to graph it, that's perfectly fine. In these cases the threshold
part of the performance data should be empty.


- --
Thomas
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.7 (MingW32)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFIKFA/6dZ+Kt5BchYRAlpjAJ9SeJYOyB5sIWO97bczw6uy64ziawCeNXfB
uvCGuEwHtqc2cAX5TMHJo/M=
=8ofk
-----END PGP SIGNATURE-----

-------------------------------------------------------------------------
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone