X-Git-Url: https://git.adam-barratt.org.uk/?a=blobdiff_plain;f=dsa-nagios-checks%2Fchecks%2Fdsa-check-hpssacli;h=d10874b705df18af9daed84f3863e586898f53b4;hb=10245f35c145c9620420b92497ce8f25725925cf;hp=220d65b11943223f12bdcb26bf3019580134a193;hpb=dbd4865bfa744c0558043a0f186904cbd19847b6;p=mirror%2Fdsa-nagios.git

diff --git a/dsa-nagios-checks/checks/dsa-check-hpssacli b/dsa-nagios-checks/checks/dsa-check-hpssacli
index 220d65b..d10874b 100755
--- a/dsa-nagios-checks/checks/dsa-check-hpssacli
+++ b/dsa-nagios-checks/checks/dsa-check-hpssacli
@@ -149,6 +149,14 @@ for my $slot (sort @controllers) {
 					record('UNKNOWN');
 				}
 			}
+			if (/^ *LD Acceleration Method: (.*)$/) {
+				my $status = $1;
+				# can at least be "Controller Cache" or HP SSD Smart Path", both OK
+				if ($status eq 'All disabled') {
+					push @{$status{'Acceleration method'}}, "LD$logicaldrive disabled";
+					record('WARNING');
+				}
+			}
 		}
 	}
 
@@ -245,7 +253,7 @@ for my $slot (sort @controllers) {
 		} elsif ($type eq 'SAS' || $type eq 'SAS+') {
 			$key = 'PHY Transfer Rate';
 			if ($value{'Interface Type'} eq 'SATA') {
-				$expected = [ '1.5Gbps', '3.0Gbps' ];
+				$expected = [ '1.5Gbps', '3.0Gbps', '6.0Gbps' ];
 			} elsif ($value{'PHY Count'} eq '2') {
 				if (defined($value{'Redundant Path(s)'})) {
 					$expected = [ '3.0GBPS, 3.0GBPS', '6.0GBPS, 6.0GBPS',
@@ -300,9 +308,18 @@ for my $slot (sort @controllers) {
 		if (/^ *(.*) Status: (.*)$/) {
 			my $system = $1;
 			my $status = $2;
+
+			if ($system eq 'Cache') {
+				# Can be:
+				# - 'OK'
+				# - 'Not Configured' (for e.g. HP SSD Smart Path)
+				# - 'Permanently Disabled'
+				# - ...?
+				next if $status =~ /^(OK|Not Configured)$/;
+			}
+
 			push @{$status{$status}}, $system;
 			if ($status ne 'OK') {
-				next if ($params->{'no-battery'} && $system eq 'Cache');
 				next if ($params->{'no-battery'} && $system eq 'Battery/Capacitor');
 				record('WARNING');
 			};