check_snmp now considers strings returned by SNMP that contain just

[nagiosplug.git] / contrib / check_linux_raid.pl

diff --git a/contrib/check_linux_raid.pl b/contrib/check_linux_raid.pl
index c7c9c55e4b2e684e6a60f878c9556814e5daf298..77e75f6d0aa787890944d0a6a1bc19b219689c0b 100644 (file)
--- a/contrib/check_linux_raid.pl
+++ b/contrib/check_linux_raid.pl
@@ -64,13 +64,15 @@ while(defined $nextdev){
                        } elsif (/recovery = (.*?)\s/) {  
                                $recovery{$device} = $1;
                                ($finish{$device}) = /finish=(.*?min)/;
+                               $device=undef;
                        } elsif (/^\s*$/) {
                                $device=undef;
                        }
                } elsif (/^($nextdev)\s*:/) {
                        $device=$1;
                        $devices{$device}=$device;
-                       if (/active/) {
+                       if (/\sactive/) {
+                               $status{$device} = ''; # Shall be filled later if available
                                $active{$device} = 1;
                        }
                }
@@ -79,7 +81,11 @@ while(defined $nextdev){
 }
 
 foreach my $k (sort keys %devices){
-       if ($status{$k} =~ /_/) {
+       if (!exists($status{$k})) {
+               $msg .= sprintf " %s inactive with no status information.",
+                       $devices{$k};
+               $code = max_state($code, "CRITICAL");
+       } elsif ($status{$k} =~ /_/) {
                if (defined $recovery{$k}) {
                        $msg .= sprintf " %s status=%s, recovery=%s, finish=%s.",
                                $devices{$k}, $status{$k}, $recovery{$k}, $finish{$k};
@@ -93,10 +99,11 @@ foreach my $k (sort keys %devices){
                $code = max_state($code, "OK");
        } else {
                if ($active{$k}) {
-                       $msg .= sprintf " %s active with no status information.\n",
+                       $msg .= sprintf " %s active with no status information.",
                                $devices{$k};
                        $code = max_state($code, "OK");
                } else {
+                       # This should't run anymore, but is left as a catch-all
                        $msg .= sprintf " %s does not exist.\n", $devices{$k};
                        $code = max_state($code, "CRITICAL");
                }