[Nagiosplug-devel] check_ntp

[Thomas Guyot-Sionnest]

> -----Original Message-----
> From: nagiosplug-devel-bounces at lists.sourceforge.net 
> [mailto:nagiosplug-devel-bounces at lists.sourceforge.net] On 
> Behalf Of Holger Weiss
> Sent: April 5, 2007 11:37
> To: Nagios Plugins Development
> Subject: Re: [Nagiosplug-devel] check_ntp
> 
> * Thomas Guyot-Sionnest <dermoth at aei.ca> [2007-04-05 09:29]:
> > On 05/04/07 03:08 AM, sean finney wrote:
> > > ah, i see.  so the control packet data probably won't 
> change in the
> > > interval that it's being checked.  in that case it 
> doesn't make any
> > > sense to average it, i agree.  however, since this is udp 
> we're talking
> > > about, maybe it's still worthwhile to throw a couple 
> extra packets on
> > > the wire to make sure one of them is recieved?  or 
> perhaps we could
> > > default to a single packet, but provide a configurable 
> retry parameter
> > > or something.
> >
> > I'll take a look at what the other check used to do... But 
> I doubt it
> > would be useful for the jitter section.
> 
> Why not?  Retries are a good idea IMO.

I haven't really tested it but with the current code I expect any packet
drop will cause the plugin to timeout. That's not really what I call
retries. So unless you have a suggestion on how to timeout on undeceived
packets (ex. Storing the time on the linked list (or whatever else) of #5
and somehow stop listening after some timeout, obviously < than the main
timeout) that could work, otherwise I'll hold my position on this.

Hey, is Ton aware that we're onboard for 1.4.8?

Thomas
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/x-pkcs7-signature
Size: 3076 bytes
Desc: not available
URL: <http://nagios-plugins.org/archive/devel/attachments/20070405/50f5eb5d/attachment.bin>