From b8126497a3b2c73612a3e4b7c73f5ba2f6a6cf8a Mon Sep 17 00:00:00 2001 From: garnier-quentin Date: Mon, 2 May 2016 21:21:15 +0200 Subject: [PATCH] + enhance output for aruba (can manage an error if no ap associated) --- .../centreon/common/airespace/snmp/mode/apstatus.pm | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/centreon-plugins/centreon/common/airespace/snmp/mode/apstatus.pm b/centreon-plugins/centreon/common/airespace/snmp/mode/apstatus.pm index 0034f233a..9fca60b16 100644 --- a/centreon-plugins/centreon/common/airespace/snmp/mode/apstatus.pm +++ b/centreon-plugins/centreon/common/airespace/snmp/mode/apstatus.pm @@ -149,7 +149,7 @@ sub check_options { sub skip_global { my ($self, %options) = @_; - scalar(keys %{$self->{ap}}) > 1 ? return(0) : return(1); + scalar(keys %{$self->{ap}}) == 1 ? return(1) : return(0); } sub prefix_ap_output { @@ -208,7 +208,7 @@ sub manage_selection { my $result3 = $options{snmp}->map_instance(mapping => $mapping3, results => $self->{results}->{ $mapping3->{bsnAPAdminStatus}->{oid} }, instance => $instance); if (defined($self->{option_results}->{filter_name}) && $self->{option_results}->{filter_name} ne '' && $result->{bsnAPName} !~ /$self->{option_results}->{filter_name}/) { - $self->{output}->output_add(long_msg => "Skipping '" . $result->{bsnAPName} . "': no matching filter.", debug => 1); + $self->{output}->output_add(long_msg => "skipping '" . $result->{bsnAPName} . "': no matching filter.", debug => 1); next; } @@ -220,8 +220,7 @@ sub manage_selection { } if (scalar(keys %{$self->{ap}}) <= 0) { - $self->{output}->output_add(severity => 'OK', - short_msg => 'No AP associated (can be: slave wireless controller or your filter)'); + $self->{output}->output_add(long_msg => 'no AP associated (can be: slave wireless controller or your filter)'); } }