[Nagiosplug-devel] [ nagiosplug-Bugs-1481019 ] Parsing problem within check_ntp

[SourceForge.net]

Bugs item #1481019, was opened at 2006-05-03 12:35
Message generated for change (Comment added) made by tonvoon
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=397597&aid=1481019&group_id=29880

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Parsing problem
Group: None
>Status: Closed
Resolution: None
Priority: 5
Submitted By: Tobias Mucke (mucke)
Assigned to: Ton Voon (tonvoon)
Summary: Parsing problem within check_ntp 

Initial Comment:
Hallo everybody,

I have a little problem with check_ntp. I can work
around it by redirecting Stderrout to /dev/null but
since this problem exists also in the newest Plugin
release 1.4.3 I want to submit this request. Here is
what I get when calling check_ntp. Lines are numbered
by me!

Line 1: Argument "(not parsed)" isn't numeric in abs at
/opt/nagios/plugins/check_ntp line 408.
Line 2: NTP OK: Offset 0.003125 secs, jitter (not
parsed) msec, peer is stratum 1|offset=0.003125,
jitter=0,peer_stratum=1

Thanks in advance.

Tobias


----------------------------------------------------------------------

>Comment By: Ton Voon (tonvoon)
Date: 2006-07-24 11:01

Message:
Logged In: YES 
user_id=664364

Roy,

The C version will be released in the upcoming 1.4.4, which is around two weeks 
away.

I'll close this call.

Ton

----------------------------------------------------------------------

Comment By: Roy-Magne Mo (gaupe)
Date: 2006-07-24 10:45

Message:
Logged In: YES 
user_id=390568

I tried the check_ntp from nagios-plugins-HEAD-200607231152,
this works like a charm for my small test case at least. Is
this version supposed to be released in 1.44?

If then I guess it's ok to carry a small patch in the fedora
package until then.

----------------------------------------------------------------------

Comment By: Ton Voon (tonvoon)
Date: 2006-07-24 09:39

Message:
Logged In: YES 
user_id=664364

Gaupe,

Can you try the check_ntp.c in the snapshot? You should go through the 
usual ./configure && make. check_ntp.pl is going to be deprecated in favour 
of the C version.

I've marked the call into pending to autoclose after 7 days. If this is still a 
problem with the C version, please reopen with details.

Ton

----------------------------------------------------------------------

Comment By: Roy-Magne Mo (gaupe)
Date: 2006-07-23 19:50

Message:
Logged In: YES 
user_id=390568

One line should be removed from check_ntp.pl, since a header
probably introduced in the output in 'ntpq -np' lately
triggers this bug:

--- check_ntp.pl.orig   2006-07-23 20:10:03.000000000 +0200
+++ check_ntp.pl        2006-07-23 20:48:08.000000000 +0200
@@ -314,7 +314,6 @@
                                }
                        } else {
                                print "No match!\n" if $verbose;
-                               $jitter = '(not parsed)';
                        }

                }


----------------------------------------------------------------------

Comment By: Roy-Magne Mo (gaupe)
Date: 2006-07-23 18:47

Message:
Logged In: YES 
user_id=390568

This problem still seems to exists in HEAD, since the
version in head and 1.43 is 1.31

----------------------------------------------------------------------

Comment By: Tobias Mucke (mucke)
Date: 2006-05-18 10:33

Message:
Logged In: YES 
user_id=481892

Hi everybody,

tried out newest snapshot release 20060517. Bug seems to be
fixed.

Closing this request.


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=397597&aid=1481019&group_id=29880