mirror of
https://github.com/monitoring-plugins/monitoring-plugins.git
synced 2026-05-28 04:35:40 -04:00
Revert check_disk performance data back to used space (#2243)
* Implement simple output shortcut for ranges If ranges start with zero (e.g. 0:10), the zero and the colon can be left out. This patch implements this by default, since some systems (icinga2) do not fully implement the whole range format and this reduces errors in the common case of just an upper border. * switch check_disk perfdata back to used space
This commit is contained in:
Lorenz Kästle
GitHub
parent
a71ce15308
commit
c57381d789
3 changed files with 169 additions and 111 deletions
|
|
@ -251,7 +251,16 @@ char *mp_range_to_string(const mp_range input) {
|
|||
if (input.start_infinity) {
|
||||
asprintf(&result, "%s~:", result);
|
||||
} else {
|
||||
asprintf(&result, "%s%s:", result, pd_value_to_string(input.start));
|
||||
// check for zeroes, so we can use the short form
|
||||
if ((input.start.type == PD_TYPE_NONE) ||
|
||||
((input.start.type == PD_TYPE_INT) && (input.start.pd_int == 0)) ||
|
||||
((input.start.type == PD_TYPE_UINT) && (input.start.pd_uint == 0)) ||
|
||||
((input.start.type == PD_TYPE_DOUBLE) && (input.start.pd_double == 0))){
|
||||
// nothing to do here
|
||||
} else {
|
||||
// Start value is an actual value
|
||||
asprintf(&result, "%s%s:", result, pd_value_to_string(input.start));
|
||||
}
|
||||
}
|
||||
|
||||
if (!input.end_infinity) {
|
||||
|
|
|
|||
|
|
@ -933,31 +933,43 @@ void set_all_thresholds(parameter_list_elem *path, char *warn_freespace_units,
|
|||
|
||||
if (warn_freespace_units) {
|
||||
tmp = mp_parse_range_string(warn_freespace_units);
|
||||
tmp.range.start = mp_create_pd_value(0);
|
||||
tmp.range.start_infinity = false;
|
||||
path->freespace_units = mp_thresholds_set_warn(path->freespace_units, tmp.range);
|
||||
}
|
||||
|
||||
if (crit_freespace_units) {
|
||||
tmp = mp_parse_range_string(crit_freespace_units);
|
||||
tmp.range.start = mp_create_pd_value(0);
|
||||
tmp.range.start_infinity = false;
|
||||
path->freespace_units = mp_thresholds_set_crit(path->freespace_units, tmp.range);
|
||||
}
|
||||
|
||||
if (warn_freespace_percent) {
|
||||
tmp = mp_parse_range_string(warn_freespace_percent);
|
||||
tmp.range.start = mp_create_pd_value(0);
|
||||
tmp.range.start_infinity = false;
|
||||
path->freespace_percent = mp_thresholds_set_warn(path->freespace_percent, tmp.range);
|
||||
}
|
||||
|
||||
if (crit_freespace_percent) {
|
||||
tmp = mp_parse_range_string(crit_freespace_percent);
|
||||
tmp.range.start = mp_create_pd_value(0);
|
||||
tmp.range.start_infinity = false;
|
||||
path->freespace_percent = mp_thresholds_set_crit(path->freespace_percent, tmp.range);
|
||||
}
|
||||
|
||||
if (warn_freeinodes_percent) {
|
||||
tmp = mp_parse_range_string(warn_freeinodes_percent);
|
||||
tmp.range.start = mp_create_pd_value(0);
|
||||
tmp.range.start_infinity = false;
|
||||
path->freeinodes_percent = mp_thresholds_set_warn(path->freeinodes_percent, tmp.range);
|
||||
}
|
||||
|
||||
if (crit_freeinodes_percent) {
|
||||
tmp = mp_parse_range_string(crit_freeinodes_percent);
|
||||
tmp.range.start = mp_create_pd_value(0);
|
||||
tmp.range.start_infinity = false;
|
||||
path->freeinodes_percent = mp_thresholds_set_crit(path->freeinodes_percent, tmp.range);
|
||||
}
|
||||
}
|
||||
|
|
@ -1182,14 +1194,22 @@ mp_subcheck evaluate_filesystem(measurement_unit measurement_unit, bool display_
|
|||
humanize_byte_value((unsigned long long)measurement_unit.total_bytes, false));
|
||||
}
|
||||
|
||||
mp_perfdata used_space = perfdata_init();
|
||||
used_space.label = measurement_unit.name;
|
||||
used_space.value = mp_create_pd_value(measurement_unit.free_bytes);
|
||||
used_space = mp_set_pd_max_value(used_space, mp_create_pd_value(measurement_unit.total_bytes));
|
||||
used_space = mp_set_pd_min_value(used_space, mp_create_pd_value(0));
|
||||
used_space.uom = "B";
|
||||
used_space = mp_pd_set_thresholds(used_space, measurement_unit.freespace_bytes_thresholds);
|
||||
freespace_bytes_sc = mp_set_subcheck_state(freespace_bytes_sc, mp_get_pd_status(used_space));
|
||||
// Free space just internally for computation
|
||||
mp_perfdata free_space_pd = perfdata_init();
|
||||
free_space_pd.label = measurement_unit.name;
|
||||
free_space_pd.value = mp_create_pd_value(measurement_unit.free_bytes);
|
||||
free_space_pd =
|
||||
mp_pd_set_thresholds(free_space_pd, measurement_unit.freespace_bytes_thresholds);
|
||||
freespace_bytes_sc = mp_set_subcheck_state(freespace_bytes_sc, mp_get_pd_status(free_space_pd));
|
||||
|
||||
// Used space for display
|
||||
mp_perfdata used_space_pd = perfdata_init();
|
||||
used_space_pd.label = measurement_unit.name;
|
||||
used_space_pd.value = mp_create_pd_value(measurement_unit.used_bytes);
|
||||
used_space_pd =
|
||||
mp_set_pd_max_value(used_space_pd, mp_create_pd_value(measurement_unit.total_bytes));
|
||||
used_space_pd = mp_set_pd_min_value(used_space_pd, mp_create_pd_value(0));
|
||||
used_space_pd.uom = "B";
|
||||
|
||||
// special case for absolute space thresholds here:
|
||||
// if absolute values are not set, compute the thresholds from percentage thresholds
|
||||
|
|
@ -1209,7 +1229,8 @@ mp_subcheck evaluate_filesystem(measurement_unit measurement_unit, bool display_
|
|||
}
|
||||
measurement_unit.freespace_bytes_thresholds =
|
||||
mp_thresholds_set_crit(measurement_unit.freespace_bytes_thresholds, tmp_range);
|
||||
used_space = mp_pd_set_thresholds(used_space, measurement_unit.freespace_bytes_thresholds);
|
||||
free_space_pd =
|
||||
mp_pd_set_thresholds(free_space_pd, measurement_unit.freespace_bytes_thresholds);
|
||||
}
|
||||
|
||||
if (!temp_thlds.warning_is_set &&
|
||||
|
|
@ -1225,10 +1246,22 @@ mp_subcheck evaluate_filesystem(measurement_unit measurement_unit, bool display_
|
|||
}
|
||||
measurement_unit.freespace_bytes_thresholds =
|
||||
mp_thresholds_set_warn(measurement_unit.freespace_bytes_thresholds, tmp_range);
|
||||
used_space = mp_pd_set_thresholds(used_space, measurement_unit.freespace_bytes_thresholds);
|
||||
free_space_pd =
|
||||
mp_pd_set_thresholds(free_space_pd, measurement_unit.freespace_bytes_thresholds);
|
||||
}
|
||||
|
||||
mp_add_perfdata_to_subcheck(&freespace_bytes_sc, used_space);
|
||||
mp_thresholds thr_used_space = measurement_unit.freespace_bytes_thresholds;
|
||||
if (thr_used_space.critical_is_set) {
|
||||
thr_used_space.critical.alert_on_inside_range =
|
||||
!thr_used_space.critical.alert_on_inside_range;
|
||||
}
|
||||
if (thr_used_space.warning_is_set) {
|
||||
thr_used_space.warning.alert_on_inside_range =
|
||||
!thr_used_space.warning.alert_on_inside_range;
|
||||
}
|
||||
used_space_pd = mp_pd_set_thresholds(used_space_pd, thr_used_space);
|
||||
|
||||
mp_add_perfdata_to_subcheck(&freespace_bytes_sc, used_space_pd);
|
||||
mp_add_subcheck_to_subcheck(&result, freespace_bytes_sc);
|
||||
|
||||
// ==========================
|
||||
|
|
@ -1264,7 +1297,8 @@ mp_subcheck evaluate_filesystem(measurement_unit measurement_unit, bool display_
|
|||
|
||||
mp_perfdata inode_percentage_pd = perfdata_init();
|
||||
inode_percentage_pd = mp_set_pd_value(inode_percentage_pd, free_inode_percentage);
|
||||
inode_percentage_pd = mp_pd_set_thresholds(inode_percentage_pd, measurement_unit.freeinodes_percent_thresholds);
|
||||
inode_percentage_pd = mp_pd_set_thresholds(inode_percentage_pd,
|
||||
measurement_unit.freeinodes_percent_thresholds);
|
||||
|
||||
if (verbose > 0) {
|
||||
printf("free inode percentage computed: %g\n", free_inode_percentage);
|
||||
|
|
|
|||
|
|
@ -7,6 +7,7 @@
|
|||
# TODO: Add in tests for perf data. Need to beef up Monitoring::Plugin::Performance to cater for max, min, etc
|
||||
|
||||
use strict;
|
||||
use warnings;
|
||||
use Test::More;
|
||||
use NPTest;
|
||||
use POSIX qw(ceil floor);
|
||||
|
|
@ -26,7 +27,7 @@ my $output_format = "--output-format mp-test-json";
|
|||
if ($mountpoint_valid eq "" or $mountpoint2_valid eq "") {
|
||||
plan skip_all => "Need 2 mountpoints to test";
|
||||
} else {
|
||||
plan tests => 97;
|
||||
plan tests => 96;
|
||||
}
|
||||
|
||||
$result = NPTest->testCmd(
|
||||
|
|
@ -34,25 +35,30 @@ $result = NPTest->testCmd(
|
|||
);
|
||||
cmp_ok( $result->return_code, "==", 0, "Checking two mountpoints (must have at least 1% free in space and inodes)");
|
||||
|
||||
my $result_mp2 = $result->{'mp_test_result'}->{'checks'}->[0];
|
||||
my $result_mp1 = $result->{'mp_test_result'}->{'checks'}->[1];
|
||||
|
||||
like($result->{'mp_test_result'}->{'state'}, "/OK/", "Main result is OK");
|
||||
like($result->{'mp_test_result'}->{'checks'}->[0]->{'state'}, "/OK/", "First sub result is OK");
|
||||
like($result->{'mp_test_result'}->{'checks'}->[1]->{'state'}, "/OK/", "Second sub result is OK");
|
||||
|
||||
my $absolut_space_mp1 = $result->{'mp_test_result'}->{'checks'}->[1]->{'checks'}->[0]->{'perfdata'}->[0]->{'max'}->{'value'};
|
||||
# print("absolute space on mp1: ". $absolut_space_mp1 . "\n");
|
||||
|
||||
my $free_percent_on_mp1 = ($result->{'mp_test_result'}->{'checks'}->[1]->{'checks'}->[0]->{'perfdata'}->[0]->{'value'}->{'value'} / ($absolut_space_mp1/100));
|
||||
print("free percent on mp1: ". $free_percent_on_mp1 . "\n");
|
||||
|
||||
my $absolut_space_mp2 = $result->{'mp_test_result'}->{'checks'}->[0]->{'checks'}->[0]->{'perfdata'}->[0]->{'max'}->{'value'};
|
||||
# print("absolute space on mp2: ". $absolut_space_mp2 . "\n");
|
||||
|
||||
my $free_percent_on_mp2 = ($result->{'mp_test_result'}->{'checks'}->[0]->{'checks'}->[0]->{'perfdata'}->[0]->{'value'}->{'value'}/ ($absolut_space_mp2/100));
|
||||
print("free percent on mp2: ". $free_percent_on_mp2 . "\n");
|
||||
like($result_mp2->{'state'}, "/OK/", "First sub result is OK");
|
||||
like($result_mp1->{'state'}, "/OK/", "Second sub result is OK");
|
||||
|
||||
my @perfdata;
|
||||
@perfdata[0] = $result->{'mp_test_result'}->{'checks'}->[0]->{'checks'}->[0]->{'perfdata'}->[0];
|
||||
@perfdata[1] = $result->{'mp_test_result'}->{'checks'}->[1]->{'checks'}->[0]->{'perfdata'}->[0];
|
||||
@perfdata[0] = $result_mp2->{'checks'}->[0]->{'perfdata'}->[0];
|
||||
@perfdata[1] = $result_mp1->{'checks'}->[0]->{'perfdata'}->[0];
|
||||
|
||||
my $absolut_space_mp1 = $perfdata[1]->{'max'}->{'value'};
|
||||
# print("absolute space on mp1: ". $absolut_space_mp1 . "\n");
|
||||
my $absolut_used_space_mp1 = $perfdata[1]->{'value'}->{'value'};
|
||||
|
||||
my $free_percent_on_mp1 = ($absolut_space_mp1 - $absolut_used_space_mp1) / ($absolut_space_mp1/100);
|
||||
# print("free percent on mp1: ". $free_percent_on_mp1 . "\n");
|
||||
|
||||
my $absolut_space_mp2 = $perfdata[0]->{'max'}->{'value'};
|
||||
# print("absolute space on mp2: ". $absolut_space_mp2 . "\n");
|
||||
my $absolut_used_space_mp2 = $perfdata[0]->{'value'}->{'value'};
|
||||
|
||||
my $free_percent_on_mp2 = (($absolut_space_mp2 - $absolut_used_space_mp2)/ ($absolut_space_mp2/100));
|
||||
# print("free percent on mp2: ". $free_percent_on_mp2 . "\n");
|
||||
|
||||
# Decrease precision of numbers since the the fs might be modified between the two runs
|
||||
$perfdata[0]->{'value'}->{'value'} = int($perfdata[0]->{'value'}->{'value'} / 1000000);
|
||||
|
|
@ -73,46 +79,56 @@ if ($free_percent_on_mp1 > $free_percent_on_mp2) {
|
|||
die "Two mountpoints are the same - cannot do rest of test";
|
||||
}
|
||||
|
||||
print("less free: " . $less_free . "\n");
|
||||
print("more free: " . $more_free . "\n");
|
||||
# print("less free: " . $less_free . "\n");
|
||||
# print("more free: " . $more_free . "\n");
|
||||
|
||||
if($free_percent_on_mp1 == $avg_free_percent || $free_percent_on_mp2 == $avg_free_percent) {
|
||||
die "One mountpoints has average space free - cannot do rest of test";
|
||||
}
|
||||
|
||||
my $used_inodes_on_mp1 = $result->{'mp_test_result'}->{'checks'}->[1]->{'checks'}[2]->{'perfdata'}->[0]->{'value'}->{'value'};
|
||||
my $total_inodes_on_mp1 = $result->{'mp_test_result'}->{'checks'}->[1]->{'checks'}[2]->{'perfdata'}->[0]->{'max'}->{'value'};
|
||||
# TODO enable inode checks later when there is enough nerves for Perl
|
||||
# my $have_inodes = 1;
|
||||
# my $more_inode_free;
|
||||
# my $less_inode_free;
|
||||
# my $avg_inode_free_percentage;
|
||||
|
||||
my $free_inodes_on_mp1 = $total_inodes_on_mp1 - $used_inodes_on_mp1;
|
||||
my $free_inode_percentage_on_mp1 = $free_inodes_on_mp1 / ($total_inodes_on_mp1 / 100);
|
||||
# # Do we have an inode reading? might not be if the filesystem does not have a fixed number of inodes
|
||||
# if ($result->{'mp_test_result'}->{'checks'}->[1]->{'checks'}[2] && $result->{'mp_test_result'}->{'checks'}->[0]) {
|
||||
# my $used_inodes_on_mp1 = $result->{'mp_test_result'}->{'checks'}->[1]->{'checks'}[2]->{'perfdata'}->[0]->{'value'}->{'value'};
|
||||
# my $total_inodes_on_mp1 = $result->{'mp_test_result'}->{'checks'}->[1]->{'checks'}[2]->{'perfdata'}->[0]->{'max'}->{'value'};
|
||||
|
||||
# print("free inodes on mp1: " . $free_inodes_on_mp1 . "\n");
|
||||
# print("total inodes on mp1: " . $total_inodes_on_mp1 . "\n");
|
||||
# print("free inode percentage on mp1: " . $free_inode_percentage_on_mp1 . "\n");
|
||||
# my $free_inodes_on_mp1 = $total_inodes_on_mp1 - $used_inodes_on_mp1;
|
||||
# my $free_inode_percentage_on_mp1 = $free_inodes_on_mp1 / ($total_inodes_on_mp1 / 100);
|
||||
|
||||
my $used_inodes_on_mp2 = $result->{'mp_test_result'}->{'checks'}->[0]->{'checks'}[2]->{'perfdata'}->[0]->{'value'}->{'value'};
|
||||
my $total_inodes_on_mp2 = $result->{'mp_test_result'}->{'checks'}->[0]->{'checks'}[2]->{'perfdata'}->[0]->{'max'}->{'value'};
|
||||
my $free_inodes_on_mp2 = $total_inodes_on_mp2 - $used_inodes_on_mp2;
|
||||
my $free_inode_percentage_on_mp2 = $free_inodes_on_mp2 / ($total_inodes_on_mp2 / 100);
|
||||
# # print("free inodes on mp1: " . $free_inodes_on_mp1 . "\n");
|
||||
# # print("total inodes on mp1: " . $total_inodes_on_mp1 . "\n");
|
||||
# # print("free inode percentage on mp1: " . $free_inode_percentage_on_mp1 . "\n");
|
||||
|
||||
# print("free inodes on mp2: " . $free_inodes_on_mp2 . "\n");
|
||||
# print("total inodes on mp2: " . $total_inodes_on_mp2 . "\n");
|
||||
# print("free inode percentage on mp2: " . $free_inode_percentage_on_mp2 . "\n");
|
||||
# my $used_inodes_on_mp2 = $result->{'mp_test_result'}->{'checks'}->[0]->{'checks'}[2]->{'perfdata'}->[0]->{'value'}->{'value'};
|
||||
# my $total_inodes_on_mp2 = $result->{'mp_test_result'}->{'checks'}->[0]->{'checks'}[2]->{'perfdata'}->[0]->{'max'}->{'value'};
|
||||
# my $free_inodes_on_mp2 = $total_inodes_on_mp2 - $used_inodes_on_mp2;
|
||||
# my $free_inode_percentage_on_mp2 = $free_inodes_on_mp2 / ($total_inodes_on_mp2 / 100);
|
||||
|
||||
my $avg_inode_free_percentage = ceil(($free_inode_percentage_on_mp1 + $free_inode_percentage_on_mp2)/2);
|
||||
my ($more_inode_free, $less_inode_free);
|
||||
if ($free_inode_percentage_on_mp1 > $free_inode_percentage_on_mp2) {
|
||||
$more_inode_free = $mountpoint_valid;
|
||||
$less_inode_free = $mountpoint2_valid;
|
||||
} elsif ($free_inode_percentage_on_mp1 < $free_inode_percentage_on_mp2) {
|
||||
$more_inode_free = $mountpoint2_valid;
|
||||
$less_inode_free = $mountpoint_valid;
|
||||
} else {
|
||||
die "Two mountpoints with same inodes free - cannot do rest of test";
|
||||
}
|
||||
if($free_inode_percentage_on_mp1 == $avg_inode_free_percentage || $free_inode_percentage_on_mp2 == $avg_inode_free_percentage) {
|
||||
die "One mountpoints has average inodes free - cannot do rest of test";
|
||||
}
|
||||
# # print("free inodes on mp2: " . $free_inodes_on_mp2 . "\n");
|
||||
# # print("total inodes on mp2: " . $total_inodes_on_mp2 . "\n");
|
||||
# # print("free inode percentage on mp2: " . $free_inode_percentage_on_mp2 . "\n");
|
||||
|
||||
# my $avg_inode_free_percentage = ceil(($free_inode_percentage_on_mp1 + $free_inode_percentage_on_mp2)/2);
|
||||
# if ($free_inode_percentage_on_mp1 > $free_inode_percentage_on_mp2) {
|
||||
# $more_inode_free = $mountpoint_valid;
|
||||
# $less_inode_free = $mountpoint2_valid;
|
||||
# } elsif ($free_inode_percentage_on_mp1 < $free_inode_percentage_on_mp2) {
|
||||
# $more_inode_free = $mountpoint2_valid;
|
||||
# $less_inode_free = $mountpoint_valid;
|
||||
# } else {
|
||||
# die "Two mountpoints with same inodes free - cannot do rest of test";
|
||||
# }
|
||||
# if($free_inode_percentage_on_mp1 == $avg_inode_free_percentage || $free_inode_percentage_on_mp2 == $avg_inode_free_percentage) {
|
||||
# die "One mountpoints has average inodes free - cannot do rest of test";
|
||||
# }
|
||||
# } else {
|
||||
# $have_inodes = 0;
|
||||
# }
|
||||
|
||||
# Verify performance data
|
||||
# First check absolute thresholds...
|
||||
|
|
@ -144,8 +160,8 @@ my $warn_percth_data = $result->{'mp_test_result'}->{'checks'}->[0]->{'checks'}[
|
|||
my $crit_percth_data = $result->{'mp_test_result'}->{'checks'}->[0]->{'checks'}[0]->{'perfdata'}->[0]->{'crit'}->{'end'}->{'value'};
|
||||
my $total_percth_data = $result->{'mp_test_result'}->{'checks'}->[0]->{'checks'}[0]->{'perfdata'}->[0]->{'max'}->{'value'};
|
||||
|
||||
print("warn_percth_data: " . $warn_percth_data . "\n");
|
||||
print("crit_percth_data: " . $crit_percth_data . "\n");
|
||||
# print("warn_percth_data: " . $warn_percth_data . "\n");
|
||||
# print("crit_percth_data: " . $crit_percth_data . "\n");
|
||||
|
||||
is (int($warn_percth_data), int((20/100)*$total_percth_data), "Wrong warning in perf data using percent thresholds. Got " . $warn_percth_data . " with total " . $total_percth_data);
|
||||
is (int($crit_percth_data), int((10/100)*$total_percth_data), "Wrong critical in perf data using percent thresholds. Got " . $crit_percth_data . " with total " . $total_percth_data);
|
||||
|
|
@ -161,6 +177,18 @@ cmp_ok( $result->return_code, "==", 0, "with JSON test format result should alwa
|
|||
my @perfdata2;
|
||||
@perfdata2[0] = $result->{'mp_test_result'}->{'checks'}->[1]->{'checks'}->[0]->{'perfdata'}->[0];
|
||||
@perfdata2[1] = $result->{'mp_test_result'}->{'checks'}->[0]->{'checks'}->[0]->{'perfdata'}->[0];
|
||||
|
||||
my $free_on_mp1 = ($perfdata2[1]->{'max'}->{'value'} - $perfdata2[1]->{'value'}->{'value'});
|
||||
my $free_on_mp2 = ($perfdata2[0]->{'max'}->{'value'} - $perfdata2[0]->{'value'}->{'value'});
|
||||
# print "free on mp1: " . $free_on_mp1 . "\n";
|
||||
# print "free on mp2: " . $free_on_mp2 . "\n";
|
||||
# Either one of those should not be zero
|
||||
die "Cannot parse output: $_" unless (defined($free_on_mp1) && defined($free_on_mp2));
|
||||
|
||||
my $free_on_all = $free_on_mp1 + $free_on_mp2;
|
||||
|
||||
# print "free on all: " . $free_on_all . "\n";
|
||||
|
||||
# Decrease precision of numbers since the the fs might be modified between the two runs
|
||||
$perfdata2[0]->{'value'}->{'value'} = int($perfdata2[0]->{'value'}->{'value'} / 1000000);
|
||||
$perfdata2[1]->{'value'}->{'value'} = int($perfdata2[1]->{'value'}->{'value'} / 1000000);
|
||||
|
|
@ -175,12 +203,6 @@ $result = NPTest->testCmd( "./check_disk -w 1 -c 1 -p $more_free -p $less_free $
|
|||
cmp_ok( $result->return_code, "==", 0, "with JSON test format result should always be OK");
|
||||
like($result->{'mp_test_result'}->{'state'}, "/OK/", "At least 1 MB available on $more_free and $less_free");
|
||||
|
||||
my $free_mb_on_mp1 =$result->{'mp_test_result'}->{'checks'}->[0]->{'checks'}->[0]->{'perfdata'}->[0]->{'value'}->{'value'} / (1024 * 1024);
|
||||
my $free_mb_on_mp2 = $result->{'mp_test_result'}->{'checks'}->[1]->{'checks'}->[0]->{'perfdata'}->[0]->{'value'}->{'value'}/ (1024 * 1024);
|
||||
die "Cannot parse output: $_" unless ($free_mb_on_mp1 && $free_mb_on_mp2);
|
||||
|
||||
my $free_mb_on_all = $free_mb_on_mp1 + $free_mb_on_mp2;
|
||||
|
||||
|
||||
$result = NPTest->testCmd( "./check_disk -e -w 1 -c 1 -p $more_free $output_format" );
|
||||
cmp_ok( $result->return_code, "==", 0, "with JSON test format result should always be OK");
|
||||
|
|
@ -248,55 +270,45 @@ cmp_ok( $result->return_code, '==', 2, "And reversing arguments should not make
|
|||
|
||||
|
||||
# Basic inode checks for sizes
|
||||
SKIP: {
|
||||
skip "No inode data", 14 if 1 eq 1;
|
||||
|
||||
$result = NPTest->testCmd( "./check_disk --icritical 1% --iwarning 1% -p $more_inode_free" );
|
||||
is( $result->return_code, 0, "At least 1% free on inodes for both mountpoints");
|
||||
# $result = NPTest->testCmd( "./check_disk --icritical 1% --iwarning 1% -p $more_inode_free" );
|
||||
# is( $result->return_code, 0, "At least 1% free on inodes for both mountpoints");
|
||||
|
||||
$result = NPTest->testCmd( "./check_disk -K 100% -W 100% -p $less_inode_free" );
|
||||
is( $result->return_code, 2, "Critical requesting 100% free inodes for both mountpoints");
|
||||
# $result = NPTest->testCmd( "./check_disk -K 100% -W 100% -p $less_inode_free" );
|
||||
# is( $result->return_code, 2, "Critical requesting 100% free inodes for both mountpoints");
|
||||
|
||||
$result = NPTest->testCmd( "./check_disk --iwarning 1% --icritical 1% -p $more_inode_free -K 100% -W 100% -p $less_inode_free" );
|
||||
is( $result->return_code, 2, "Get critical on less_inode_free mountpoint $less_inode_free");
|
||||
# $result = NPTest->testCmd( "./check_disk --iwarning 1% --icritical 1% -p $more_inode_free -K 100% -W 100% -p $less_inode_free" );
|
||||
# is( $result->return_code, 2, "Get critical on less_inode_free mountpoint $less_inode_free");
|
||||
|
||||
$result = NPTest->testCmd( "./check_disk -W $avg_inode_free_percentage% -K 0% -p $less_inode_free" );
|
||||
is( $result->return_code, 1, "Get warning on less_inode_free, when checking average");
|
||||
# $result = NPTest->testCmd( "./check_disk -W $avg_inode_free_percentage% -K 0% -p $less_inode_free" );
|
||||
# is( $result->return_code, 1, "Get warning on less_inode_free, when checking average");
|
||||
|
||||
$result = NPTest->testCmd( "./check_disk -W $avg_inode_free_percentage% -K $avg_inode_free_percentage% -p $more_inode_free ");
|
||||
is( $result->return_code, 0, "Get ok on more_inode_free when checking average");
|
||||
# $result = NPTest->testCmd( "./check_disk -W $avg_inode_free_percentage% -K $avg_inode_free_percentage% -p $more_inode_free ");
|
||||
# is( $result->return_code, 0, "Get ok on more_inode_free when checking average");
|
||||
|
||||
$result = NPTest->testCmd( "./check_disk -W $avg_inode_free_percentage% -K 0% -p $less_inode_free -W $avg_inode_free_percentage% -K $avg_inode_free_percentage% -p $more_inode_free" );
|
||||
is ($result->return_code, 1, "Combine above two tests, get warning");
|
||||
$all_disks = $result->output;
|
||||
# $result = NPTest->testCmd( "./check_disk -W $avg_inode_free_percentage% -K 0% -p $less_inode_free -W $avg_inode_free_percentage% -K $avg_inode_free_percentage% -p $more_inode_free" );
|
||||
# is ($result->return_code, 1, "Combine above two tests, get warning");
|
||||
# $all_disks = $result->output;
|
||||
|
||||
$result = NPTest->testCmd( "./check_disk -e -W $avg_inode_free_percentage% -K 0% -p $less_inode_free -W $avg_inode_free_percentage% -K $avg_inode_free_percentage% -p $more_inode_free" );
|
||||
isnt( $result->output, $all_disks, "-e gives different output");
|
||||
like( $result->output, qr/$less_inode_free/, "Found problem $less_inode_free");
|
||||
unlike( $result->only_output, qr/$more_inode_free\s/, "Has ignored $more_inode_free as not a problem");
|
||||
like( $result->perf_output, qr/$more_inode_free/, "But $more_inode_free is still in perf data");
|
||||
# $result = NPTest->testCmd( "./check_disk -e -W $avg_inode_free_percentage% -K 0% -p $less_inode_free -W $avg_inode_free_percentage% -K $avg_inode_free_percentage% -p $more_inode_free" );
|
||||
# isnt( $result->output, $all_disks, "-e gives different output");
|
||||
# like( $result->output, qr/$less_inode_free/, "Found problem $less_inode_free");
|
||||
# unlike( $result->only_output, qr/$more_inode_free\s/, "Has ignored $more_inode_free as not a problem");
|
||||
# like( $result->perf_output, qr/$more_inode_free/, "But $more_inode_free is still in perf data");
|
||||
|
||||
$result = NPTest->testCmd( "./check_disk -W $avg_inode_free_percentage% -K 0% -p $more_inode_free" );
|
||||
is( $result->return_code, 0, "Get ok on more_inode_free mountpoint, checking average");
|
||||
# $result = NPTest->testCmd( "./check_disk -W $avg_inode_free_percentage% -K 0% -p $more_inode_free" );
|
||||
# is( $result->return_code, 0, "Get ok on more_inode_free mountpoint, checking average");
|
||||
|
||||
$result = NPTest->testCmd( "./check_disk -W $avg_inode_free_percentage% -K $avg_inode_free_percentage% -p $less_inode_free" );
|
||||
is( $result->return_code, 2, "Get critical on less_inode_free, checking average");
|
||||
# $result = NPTest->testCmd( "./check_disk -W $avg_inode_free_percentage% -K $avg_inode_free_percentage% -p $less_inode_free" );
|
||||
# is( $result->return_code, 2, "Get critical on less_inode_free, checking average");
|
||||
|
||||
$result = NPTest->testCmd( "./check_disk -W $avg_inode_free_percentage% -K 0% -p $more_inode_free -W $avg_inode_free_percentage% -K $avg_inode_free_percentage% -p $less_inode_free" );
|
||||
is( $result->return_code, 2, "Combining above two tests, get critical");
|
||||
# $result = NPTest->testCmd( "./check_disk -W $avg_inode_free_percentage% -K 0% -p $more_inode_free -W $avg_inode_free_percentage% -K $avg_inode_free_percentage% -p $less_inode_free" );
|
||||
# is( $result->return_code, 2, "Combining above two tests, get critical");
|
||||
|
||||
$result = NPTest->testCmd( "./check_disk -W $avg_inode_free_percentage% -K $avg_inode_free_percentage% -p $less_inode_free -W $avg_inode_free_percentage% -K 0% -p $more_inode_free" );
|
||||
cmp_ok( $result->return_code, '==', 2, "And reversing arguments should not make a difference");
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
TODO: {
|
||||
local $TODO = "Invalid percent figures";
|
||||
$result = NPTest->testCmd(
|
||||
"./check_disk -w 10% -c 15% -p $mountpoint_valid"
|
||||
);
|
||||
cmp_ok( $result->return_code, '==', 3, "Invalid command line options" );
|
||||
# $result = NPTest->testCmd( "./check_disk -W $avg_inode_free_percentage% -K $avg_inode_free_percentage% -p $less_inode_free -W $avg_inode_free_percentage% -K 0% -p $more_inode_free" );
|
||||
# cmp_ok( $result->return_code, '==', 2, "And reversing arguments should not make a difference");
|
||||
}
|
||||
|
||||
$result = NPTest->testCmd(
|
||||
|
|
@ -371,20 +383,23 @@ $result = NPTest->testCmd( "./check_disk -w 0% -c 0% -C -w 0% -c 0% -p $mountpoi
|
|||
cmp_ok( $result->return_code, "==", 0, "with JSON test format result should always be OK");
|
||||
cmp_ok(scalar $result->{'mp_test_result'}->{'checks'}, '>', 1, "-C invokes matchall logic again");
|
||||
|
||||
my $value_below = ($free_on_all - (10**6)) % (10**5) * (10**5);
|
||||
my $value_above = $free_on_all + (10**6) % (10**5) * (10**5);
|
||||
|
||||
# grouping: exit crit if the sum of free megs on mp1+mp2 is less than warn/crit
|
||||
$result = NPTest->testCmd( "./check_disk -w ". ($free_mb_on_all + 1) ." -c ". ($free_mb_on_all + 1) ." -g group -p $mountpoint_valid -p $mountpoint2_valid" );
|
||||
cmp_ok( $result->return_code, '==', 2, "grouping: exit crit if the sum of free megs on mp1+mp2 is less than warn/crit\nInstead received: " . $result->output);
|
||||
$result = NPTest->testCmd( "./check_disk -u Bytes -w ". $value_above ." -c ". $value_above ." -g group -p $mountpoint_valid -p $mountpoint2_valid" );
|
||||
cmp_ok( $result->return_code, '==', 2, "grouping: exit crit if the sum of free megs on mp1+mp2 is less than warn/crit");
|
||||
|
||||
# grouping: exit warning if the sum of free megs on mp1+mp2 is between -w and -c
|
||||
$result = NPTest->testCmd( "./check_disk -w ". ($free_mb_on_all + 1) ." -c ". ($free_mb_on_all - 1) ." -g group -p $mountpoint_valid -p $mountpoint2_valid" );
|
||||
$result = NPTest->testCmd( "./check_disk -u Bytes -w ". $value_above ." -c ". $value_below ." -g group -p $mountpoint_valid -p $mountpoint2_valid" );
|
||||
cmp_ok( $result->return_code, '==', 1, "grouping: exit warning if the sum of free megs on mp1+mp2 is between -w and -c ");
|
||||
|
||||
# grouping: exit ok if the sum of free megs on mp1+mp2 is more than warn/crit
|
||||
$result = NPTest->testCmd( "./check_disk -w ". ($free_mb_on_all - 1) ." -c ". ($free_mb_on_all - 1) ." -g group -p $mountpoint_valid -p $mountpoint2_valid" );
|
||||
$result = NPTest->testCmd( "./check_disk -u Bytes -w ". $value_below ." -c ". $value_below ." -g group -p $mountpoint_valid -p $mountpoint2_valid" );
|
||||
cmp_ok( $result->return_code, '==', 0, "grouping: exit ok if the sum of free megs on mp1+mp2 is more than warn/crit");
|
||||
|
||||
# grouping: exit unknown if group name is given after -p
|
||||
$result = NPTest->testCmd( "./check_disk -w ". ($free_mb_on_all - 1) ." -c ". ($free_mb_on_all - 1) ." -p $mountpoint_valid -g group -p $mountpoint2_valid" );
|
||||
$result = NPTest->testCmd( "./check_disk -u Bytes -w ". $value_below ." -c ". $value_below ." -p $mountpoint_valid -g group -p $mountpoint2_valid" );
|
||||
cmp_ok( $result->return_code, '==', 3, "Invalid options: -p must come after groupname");
|
||||
|
||||
# regex: exit unknown if given regex is not compilable
|
||||
|
|
|
|||
Loading…
Reference in a new issue