From: Ton Voon Date: Fri, 11 Aug 2006 08:26:34 +0000 (+0000) Subject: Catch no responses from any server (1538341 - nmdias) X-Git-Url: https://git.tokkee.org/?a=commitdiff_plain;h=0c2be6a608135d8a45c82a1c0ada5e8020bc2409;p=nagiosplug.git Catch no responses from any server (1538341 - nmdias) git-svn-id: https://nagiosplug.svn.sourceforge.net/svnroot/nagiosplug/nagiosplug/trunk@1467 f882894a-f735-0410-b71e-b25c423dba1c --- diff --git a/plugins/check_ntp.c b/plugins/check_ntp.c index 9cda277..f9dc49a 100644 --- a/plugins/check_ntp.c +++ b/plugins/check_ntp.c @@ -335,7 +335,7 @@ int best_offset_server(const ntp_server_results *slist, int nservers){ * we have to do it in a way that our lazy macros don't handle currently :( */ double offset_request(const char *host, int *status){ int i=0, j=0, ga_result=0, num_hosts=0, *socklist=NULL, respnum=0; - int servers_completed=0, one_written=0, servers_readable=0, best_index=-1; + int servers_completed=0, one_written=0, one_read=0, servers_readable=0, best_index=-1; time_t now_time=0, start_ts=0; ntp_message *req=NULL; double avg_offset=0.; @@ -437,12 +437,17 @@ double offset_request(const char *host, int *status){ servers[i].rtdelay=NTP32asDOUBLE(req[i].rtdelay); servers[i].waiting=0; servers_readable--; + one_read = 1; if(servers[i].num_responses==AVG_NUM) servers_completed++; } } /* lather, rinse, repeat. */ } + if (one_read == 0) { + die(STATE_CRITICAL, "NTP CRITICAL: No response from NTP server\n"); + } + /* now, pick the best server from the list */ best_index=best_offset_server(servers, num_hosts); if(best_index < 0){ diff --git a/plugins/t/check_ntp.t b/plugins/t/check_ntp.t new file mode 100644 index 0000000..6ba521f --- /dev/null +++ b/plugins/t/check_ntp.t @@ -0,0 +1,57 @@ +#! /usr/bin/perl -w -I .. +# +# Testing NTP +# +# $Id$ +# + +use strict; +use Test::More; +use NPTest; + +plan tests => 4; + +my $res; + +my $ntp_service = getTestParameter( "NP_GOOD_NTP_SERVICE", + "A host providing NTP service", + "pool.ntp.org"); + +my $no_ntp_service = getTestParameter( "NP_NO_NTP_SERVICE", + "A host NOT providing the NTP service", + "localhost" ); + +my $host_nonresponsive = getTestParameter( "NP_HOST_NONRESPONSIVE", + "The hostname of system not responsive to network requests", + "10.0.0.1" ); + +my $hostname_invalid = getTestParameter( "NP_HOSTNAME_INVALID", + "An invalid (not known to DNS) hostname", + "nosuchhost"); + +SKIP: { + skip "No NTP server defined", 1 unless $ntp_service; + $res = NPTest->testCmd( + "./check_ntp -H $ntp_service" + ); + cmp_ok( $res->return_code, '==', 0, "Got good NTP result"); +} + +SKIP: { + skip "No bad NTP server defined", 1 unless $ntp_service; + $res = NPTest->testCmd( + "./check_ntp -H $no_ntp_service" + ); + cmp_ok( $res->return_code, '==', 2, "Got bad NTP result"); +} + +$res = NPTest->testCmd( + "./check_ntp -H $host_nonresponsive" + ); +cmp_ok( $res->return_code, '==', 2, "Got critical if server not responding"); + +$res = NPTest->testCmd( + "./check_ntp -H $hostname_invalid" + ); +cmp_ok( $res->return_code, '==', 3, "Got critical if server hostname invalid"); +