From a69aa93a2c0305cb2168daba66b219c7de3bebde Mon Sep 17 00:00:00 2001 From: Thomas Guyot-Sionnest Date: Mon, 29 Jan 2007 14:14:05 +0000 Subject: Revert 1643463 (reversed disk thresholds) keeping the thresholds checks. git-svn-id: https://nagiosplug.svn.sourceforge.net/svnroot/nagiosplug/nagiosplug/trunk@1594 f882894a-f735-0410-b71e-b25c423dba1c diff --git a/plugins/check_disk.c b/plugins/check_disk.c index 476c892..99a7201 100644 --- a/plugins/check_disk.c +++ b/plugins/check_disk.c @@ -306,16 +306,16 @@ main (int argc, char **argv) critical_high_tide = UINT_MAX; if (path->freespace_units->warning != NULL) { - warning_high_tide = path->freespace_units->warning->end; + warning_high_tide = dtotal_units - path->freespace_units->warning->end; } if (path->freespace_percent->warning != NULL) { - warning_high_tide = abs( min( (double) warning_high_tide, (double) path->freespace_percent->warning->end/100*dtotal_units )); + warning_high_tide = abs( min( (double) warning_high_tide, (double) (1.0 - path->freespace_percent->warning->end/100)*dtotal_units )); } if (path->freespace_units->critical != NULL) { - critical_high_tide = path->freespace_units->critical->end; + critical_high_tide = dtotal_units - path->freespace_units->critical->end; } if (path->freespace_percent->critical != NULL) { - critical_high_tide = abs( min( (double) critical_high_tide, (double) path->freespace_percent->critical->end/100*dtotal_units )); + critical_high_tide = abs( min( (double) critical_high_tide, (double) (1.0 - path->freespace_percent->critical->end/100)*dtotal_units )); } asprintf (&perf, "%s %s", perf, diff --git a/plugins/t/check_disk.t b/plugins/t/check_disk.t index 7235d70..dd13ce5 100644 --- a/plugins/t/check_disk.t +++ b/plugins/t/check_disk.t @@ -82,9 +82,9 @@ $result = NPTest->testCmd( "./check_disk -w 20 -c 10 -p $mountpoint_valid" ); $_ = $result->perf_output; -my ($warn_absth_data, $crit_absth_data) = (m/=.[^;]*;(\d+);(\d+);\d+;\d+/); -is ($warn_absth_data, 20, "Wrong warning in perf data using absolute thresholds"); -is ($crit_absth_data, 10, "Wrong critical in perf data using absolute thresholds"); +my ($warn_absth_data, $crit_absth_data, $total_absth_data) = (m/=.[^;]*;(\d+);(\d+);\d+;(\d+)/); +is ($warn_absth_data, $total_absth_data - 20, "Wrong warning in perf data using absolute thresholds"); +is ($crit_absth_data, $total_absth_data - 10, "Wrong critical in perf data using absolute thresholds"); # Then check percent thresholds. $result = NPTest->testCmd( @@ -92,8 +92,8 @@ $result = NPTest->testCmd( ); $_ = $result->perf_output; my ($warn_percth_data, $crit_percth_data, $total_percth_data) = (m/=.[^;]*;(\d+);(\d+);\d+;(\d+)/); -is ($warn_percth_data, int($total_percth_data/100*20), "Wrong warning in perf data using percent thresholds"); -is ($crit_percth_data, int($total_percth_data/100*10), "Wrong critical in perf data using percent thresholds"); +is ($warn_percth_data, int((1-20/100)*$total_percth_data), "Wrong warning in perf data using percent thresholds"); +is ($crit_percth_data, int((1-10/100)*$total_percth_data), "Wrong critical in perf data using percent thresholds"); # Check when order of mount points are reversed, that perf data remains same -- cgit v0.10-9-g596f