[Nagiosplug-devel] Enhancement Patch for check_icmp

[Thomas Guyot-Sionnest]

On 07/03/07 01:59 PM, Thomas Guyot-Sionnest wrote:
> 
> Indeed this is an interesting feature. I'll rework it a bit and commit it
> into CVS shortly.
> 
> What I will change:
> 
> 1. minHostsOk will be set to NULL initially
> 2. minHostsOk data type does not match with strtoul()
> 3. Return WARNING if at least minHostsOk hosts are in OK or WARNING state.
> 4. Will possibly add sanity checking
> 5. Add a testcase

This is in CVS.

#1: I use -1 for unset as I expect some users may want to use 0.
#2 was me reading too fast the patch, non-issue :)
#5: There is no test folder there (Ton?)

Thanks,

Thomas