Hi Holger,<br>
<br>
FWIW I had in mind for quite some time adding an option to set the state<br>
on timeout - and that's not just for the check_ntp_* plugins... Do you<br>
think there's something fairly generic we could add to many similar<br>
plugins? Check_http, check_ntp*, check_snmp, check_nrpe are some of the<br>
plugins for which id's have had use for this sort of behavior in the<br>
past (there's probably more...).<br>
<br>
Regards,<br>
<br>
<br>
On 21/03/14 06:02 AM, Holger Weiß wrote:<br>
><br>
> Ah, having |-T| accept an argument was exactly what I had in mind,<br>
> after playing around with |negate| I had only forgotten that your<br>
> patch doesn't do just that (had no coffee yet today). So yes, that<br>
> would be great.<br>
><br>
> There's |plugins/t/check_time.t| and |plugins/t/check_ntp.t| (the<br>
> latter is testing all |check_ntp*| plugins). I can add one or two test<br>
> cases myself if it's not obvious how to do that.<br>
><br>
> Thanks!<br>
><br>
> —<br>
> Reply to this email on GitHub<br>
> <https://github.com/monitoring-plugins/monitoring-plugins/pull/1246#issuecomment-38262180>.<br>
><br>
<br>
-- <br>
Thomas

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>Reply to this email directly or <a href='https://github.com/monitoring-plugins/monitoring-plugins/pull/1246#issuecomment-40679553'>view it on GitHub</a>.<img src='https://github.com/notifications/beacon/5514713__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcxMzMyNzcyMywiZGF0YSI6eyJpZCI6Mjc2MTIzOTF9fQ==--5002ff512cda25a0e1615e22f693150bda80f4af.gif' height='1' width='1'></p>