Code

Attempt at fixing check_snmp multiline output:
[nagiosplug.git] / plugins-scripts / check_ntp.pl
index d0167cdedd019e38c3af543bb055382f5407dd5c..5c87e0a66e66ba003b9d3b6597c0d0fd086a9993 100755 (executable)
@@ -1,5 +1,5 @@
 #!/usr/bin/perl -w
-
+#
 # (c)1999 Ian Cass, Knowledge Matters Ltd.
 # Read the GNU copyright stuff for all the legalese
 #
@@ -7,7 +7,6 @@
 # be installed on the system, however since it's part of the ntp suite, you 
 # should already have it installed.
 #
-# $Id$
 #
 # Nothing clever done in this program - its a very simple bare basics hack to
 # get the job done.
@@ -97,7 +96,7 @@ GetOptions
         "H=s" => \$opt_H, "hostname=s" => \$opt_H);
 
 if ($opt_V) {
-       print_revision($PROGNAME,'$Revision$ ');
+       print_revision($PROGNAME,'@NP_VERSION@');
        exit $ERRORS{'OK'};
 }
 
@@ -200,13 +199,15 @@ elsif ($ipv6) {
 ###
 
 if (!open (NTPDATE, $ntpdate . " -q $host 2>&1 |")) {
-       print "Could not open ntpdate\n";
+       print "Could not open $ntpdate: $!\n";
        exit $ERRORS{"UNKNOWN"};
 }
 
+my $out;
 while (<NTPDATE>) {
        #print if ($verbose);  # noop
        $msg = $_ unless ($msg);
+       $out .= "$_ ";
        
        if (/stratum\s(\d+)/) {
                $stratum = $1;
@@ -241,8 +242,11 @@ while (<NTPDATE>) {
        }
 
 }
+$out =~ s/\n//g;
+close (NTPDATE) || 
+    die $! ? "$out - Error closing $ntpdate pipe: $!"
+           : "$out - Exit status: $? from $ntpdate\n";
 
-close (NTPDATE); 
 # declare an error if we also get a non-zero return code from ntpdate
 # unless already set to critical
 if ( $? && !$ignoreret ) {
@@ -289,11 +293,11 @@ if ($have_ntpq) {
                        if (/^(\*|\+|\#|o])/) {
                                ++$candidate;
                                push (@candidates, $_);
-                               print "Candiate count= $candidate\n" if ($verbose);
+                               print "Candidate count= $candidate\n" if ($verbose);
                        }
                        
                        # match sys.peer or pps.peer
-                       if (/^(\*|o)([-0-9.\s]+)\s+([-0-9A-Za-z_().]+)\s+([-0-9.]+)\s+([lumb-]+)\s+([-0-9hm.]+)\s+([-0-9.mh]+)\s+([-0-9.]+)\s+([-0-9.]+)\s+([-0-9.]+)\s+([-0-9.]+)/) {
+                       if (/^(\*|o)(\S+)\s+(\S+)\s+(\S+)\s+(\S+)\s+(\S+)\s+(\S+)\s+(\S+)\s+(\S+)\s+(\S+)\s+(\S+)/) {
                                $syspeer = $2;
                                $stratum = $4;
                                $jitter = $11;
@@ -302,15 +306,20 @@ if ($have_ntpq) {
                                        print "Jitter_crit = $11 :$jcrit\n" if ($verbose);
                                        $jitter_error = $ERRORS{'CRITICAL'};
                                } elsif ($jitter > $jwarn ) {
-                                       print "Jitter_warn = $11 :$jwarn \n" if ($verbose);
+                                       print "Jitter_warn = $11 :$jwarn\n" if ($verbose);
                                        $jitter_error = $ERRORS{'WARNING'};
                                } else {
                                        $jitter_error = $ERRORS{'OK'};
                                }
+                       } else {
+                               print "No match!\n" if $verbose;
+                               $jitter = '(not parsed)';
                        }
                        
                }
-               close NTPQ;
+               close NTPQ ||
+            die $! ? "Error closing $ntpq pipe: $!"
+                   : "Exit status: $? from $ntpq\n";
 
                # if we did not match sys.peer or pps.peer but matched # candidates only
                # generate a warning 
@@ -343,53 +352,53 @@ if ($have_ntpq) {
 if ($ntpdate_error != $ERRORS{'OK'}) {
        $state = $ntpdate_error;
        if ($ntpdate_error == $ERRORS{'WARNING'} ) {
-               $answer = $msg . "\n";
+               $answer = $msg;
        }
        else {
-               $answer = $msg . "Server for ntp probably down\n";
+               $answer = $msg . "Server for ntp probably down";
        }
 
        if (defined($offset) && abs($offset) > $ocrit) {
                $state = $ERRORS{'CRITICAL'};
-               $answer = "Server Error and offset $offset sec > +/- $ocrit sec\n";
+               $answer = "Server Error and offset $offset sec > +/- $ocrit sec";
        } elsif (defined($offset) && abs($offset) > $owarn) {
-               $answer = "Server error and offset $offset sec > +/- $owarn sec\n";
+               $answer = "Server error and offset $offset sec > +/- $owarn sec";
        } elsif (defined($jitter) && abs($jitter) > $jcrit) {
-               $answer = "Server error and jitter $jitter msec > +/- $jcrit msec\n";
+               $answer = "Server error and jitter $jitter msec > +/- $jcrit msec";
        } elsif (defined($jitter) && abs($jitter) > $jwarn) {
-               $answer = "Server error and jitter $jitter msec > +/- $jwarn msec\n";
+               $answer = "Server error and jitter $jitter msec > +/- $jwarn msec";
        }
 
 } elsif ($have_ntpq && $jitter_error != $ERRORS{'OK'}) {
        $state = $jitter_error;
-       $answer = "Jitter $jitter too high\n";
+       $answer = "Jitter $jitter too high";
        if (defined($offset) && abs($offset) > $ocrit) {
                $state = $ERRORS{'CRITICAL'};
-               $answer = "Jitter error and offset $offset sec > +/- $ocrit sec\n";
+               $answer = "Jitter error and offset $offset sec > +/- $ocrit sec";
        } elsif (defined($offset) && abs($offset) > $owarn) {
-               $answer = "Jitter error and offset $offset sec > +/- $owarn sec\n";
+               $answer = "Jitter error and offset $offset sec > +/- $owarn sec";
        } elsif (defined($jitter) && abs($jitter) > $jcrit) {
-               $answer = "Jitter error and jitter $jitter msec > +/- $jcrit msec\n";
+               $answer = "Jitter error and jitter $jitter msec > +/- $jcrit msec";
        } elsif (defined($jitter) && abs($jitter) > $jwarn) {
-               $answer = "Jitter error and jitter $jitter msec > +/- $jwarn msec\n";
+               $answer = "Jitter error and jitter $jitter msec > +/- $jwarn msec";
        }
 
 } elsif( !$have_ntpq ) { # no errors from ntpdate and no ntpq or ntpq timed out
        if (abs($offset) > $ocrit) {
                $state = $ERRORS{'CRITICAL'};
-               $answer = "Offset $offset sec > +/- $ocrit sec\n";
+               $answer = "Offset $offset sec > +/- $ocrit sec";
        } elsif (abs($offset) > $owarn) {
                $state = $ERRORS{'WARNING'};
-               $answer = "Offset $offset sec > +/- $owarn sec\n";
+               $answer = "Offset $offset sec > +/- $owarn sec";
        } elsif (( abs($offset) > $owarn) && $def_jitter ) {
                $state = $ERRORS{'WARNING'};
-               $answer = "Offset $offset sec > +/- $owarn sec, ntpq timed out\n";
+               $answer = "Offset $offset sec > +/- $owarn sec, ntpq timed out";
        } elsif ( $def_jitter ) {
                $state = $ERRORS{'WARNING'};
-               $answer = "Offset $offset secs, ntpq timed out\n";
+               $answer = "Offset $offset secs, ntpq timed out";
        } else{
                $state = $ERRORS{'OK'};
-               $answer = "Offset $offset secs \n";
+               $answer = "Offset $offset secs";
        }
 
 
@@ -397,27 +406,28 @@ if ($ntpdate_error != $ERRORS{'OK'}) {
 } else { # no errors from ntpdate or ntpq
        if (abs($offset) > $ocrit) {
                $state = $ERRORS{'CRITICAL'};
-               $answer = "Offset $offset sec > +/- $ocrit sec, jitter $jitter msec\n";
+               $answer = "Offset $offset sec > +/- $ocrit sec, jitter $jitter msec";
        } elsif (abs($jitter) > $jcrit ) {
                $state = $ERRORS{'CRITICAL'};
-               $answer = "Jitter $jitter msec> +/- $jcrit msec, offset $offset sec \n";
+               $answer = "Jitter $jitter msec> +/- $jcrit msec, offset $offset sec";
        } elsif (abs($offset) > $owarn) {
                $state = $ERRORS{'WARNING'};
-               $answer = "Offset $offset sec > +/- $owarn sec, jitter $jitter msec\n";
+               $answer = "Offset $offset sec > +/- $owarn sec, jitter $jitter msec";
        } elsif (abs($jitter) > $jwarn ) {
                $state = $ERRORS{'WARNING'};
-               $answer = "Jitter $jitter msec> +/- $jwarn msec, offset $offset sec \n";
+               $answer = "Jitter $jitter msec> +/- $jwarn msec, offset $offset sec";
 
        } else {
                $state = $ERRORS{'OK'};
-               $answer = "Offset $offset secs, jitter $jitter msec, peer is stratum $stratum\n";
+               $answer = "Offset $offset secs, jitter $jitter msec, peer is stratum $stratum";
        }
        
 }
 
 foreach my $key (keys %ERRORS) {
        if ($state==$ERRORS{$key}) {
-               print ("NTP $key: $answer");
+#              print ("NTP $key: $answer");
+               print ("NTP $key: $answer|offset=$offset, jitter=" . $jitter/1000 .     ",peer_stratum=$stratum\n");
                last;
        }
 }
@@ -432,7 +442,7 @@ sub print_usage () {
 }
 
 sub print_help () {
-       print_revision($PROGNAME,'$Revision$');
+       print_revision($PROGNAME,'@NP_VERSION@');
        print "Copyright (c) 2003 Bo Kersey/Karl DeBisschop\n";
        print "\n";
        print_usage();