check-health: do not hardcode names, use as available

This commit is contained in:
Christian Hesse 2020-10-17 23:05:08 +02:00
parent a4b237044d
commit 211edb93c0

View file

@ -32,54 +32,57 @@
:set CheckHealthTemperatureNotified [ :toarray "" ]; :set CheckHealthTemperatureNotified [ :toarray "" ];
} }
:foreach Voltage in={ "battery"; "psu1-voltage"; "psu2-voltage"; "voltage" } do={ :foreach Name,Voltage in=$CheckHealthCurrent do={
:if ([ :typeof ($CheckHealthLast->$Voltage) ] = "num" && \ :if ($Name ~ "(battery|voltage)" && \
[ :typeof ($CheckHealthCurrent->$Voltage) ] = "num") do={ [ :typeof ($CheckHealthLast->$Name) ] = "num" && \
:if ($CheckHealthLast->$Voltage * (100 + $CheckHealthVoltagePercent) < $CheckHealthCurrent->$Voltage * 100 || \ [ :typeof $Voltage ] = "num") do={
$CheckHealthLast->$Voltage * 100 > $CheckHealthCurrent->$Voltage * (100 + $CheckHealthVoltagePercent)) do={ :if ($CheckHealthLast->$Name * (100 + $CheckHealthVoltagePercent) < $Voltage * 100 || \
$SendNotification ([ $SymbolForNotification "high-voltage-sign" ] . "Health warning: " . $Voltage) \ $CheckHealthLast->$Name * 100 > $Voltage * (100 + $CheckHealthVoltagePercent)) do={
("The " . $Voltage . " on " . $Identity . " jumped more than " . $CheckHealthVoltagePercent . "%.\n\n" . \ $SendNotification ([ $SymbolForNotification "high-voltage-sign" ] . "Health warning: " . $Name) \
"old value: " . [ $FormatVoltage ($CheckHealthLast->$Voltage) ] . "\n" . \ ("The " . $Name . " on " . $Identity . " jumped more than " . $CheckHealthVoltagePercent . "%.\n\n" . \
"new value: " . [ $FormatVoltage ($CheckHealthCurrent->$Voltage) ]); "old value: " . [ $FormatVoltage ($CheckHealthLast->$Name) ] . "\n" . \
"new value: " . [ $FormatVoltage $Voltage ]);
} }
} }
} }
:foreach PSU in={ "psu1"; "psu2" } do={ :foreach Name,PSU in=$CheckHealthCurrent do={
:if ([ :typeof ($CheckHealthLast->($PSU . "-state")) ] = "str" && \ :if ($Name ~ "psu.*-state" && \
[ :typeof ($CheckHealthCurrent->($PSU . "-state")) ] = "str") do={ [ :typeof ($CheckHealthLast->$Name) ] = "str" && \
:if ($CheckHealthLast->($PSU . "-state") = "ok" && \ [ :typeof $PSU ] = "str") do={
$CheckHealthCurrent->($PSU . "-state") != "ok") do={ :if ($CheckHealthLast->$Name = "ok" && \
$SendNotification ([ $SymbolForNotification "cross-mark" ] . "Health warning: " . $PSU . " state") \ $PSU != "ok") do={
("The power supply unit '" . $PSU . "' on " . $Identity . " failed!"); $SendNotification ([ $SymbolForNotification "cross-mark" ] . "Health warning: " . $Name) \
("The power supply unit '" . $Name . "' on " . $Identity . " failed!");
} }
:if ($CheckHealthLast->($PSU . "-state") != "ok" && \ :if ($CheckHealthLast->$Name != "ok" && \
$CheckHealthCurrent->($PSU . "-state") = "ok") do={ $PSU = "ok") do={
$SendNotification ([ $SymbolForNotification "white-heavy-check-mark" ] . "Health recovery: " . $PSU . " state") \ $SendNotification ([ $SymbolForNotification "white-heavy-check-mark" ] . "Health recovery: " . $Name) \
("The power supply unit '" . $PSU . "' on " . $Identity . " recovered!"); ("The power supply unit '" . $Name . "' on " . $Identity . " recovered!");
} }
} }
} }
:foreach Temperature in={ "temperature"; "cpu-temperature"; "board-temperature1"; "board-temperature2" } do={ :foreach Name,Temperature in=$CheckHealthCurrent do={
:if ([ :typeof ($CheckHealthCurrent->$Temperature) ] = "num") do={ :if ($Name ~ "temperature" && \
:if ([ :typeof ($CheckHealthTemperature->$Temperature) ] != "num" ) do={ [ :typeof $Temperature ] = "num") do={
$LogPrintExit warning ("No threshold given for " . $Temperature . ", assuming 50C.") false; :if ([ :typeof ($CheckHealthTemperature->$Name) ] != "num" ) do={
:set ($CheckHealthTemperature->$Temperature) 50; $LogPrintExit warning ("No threshold given for " . $Name . ", assuming 50C.") false;
:set ($CheckHealthTemperature->$Name) 50;
} }
:if ($CheckHealthCurrent->$Temperature > $CheckHealthTemperature->$Temperature && \ :if ($Temperature > $CheckHealthTemperature->$Name && \
$CheckHealthTemperatureNotified->$Temperature != true) do={ $CheckHealthTemperatureNotified->$Name != true) do={
$SendNotification ([ $SymbolForNotification "fire" ] . "Health warning: " . $Temperature) \ $SendNotification ([ $SymbolForNotification "fire" ] . "Health warning: " . $Name) \
("The " . $Temperature . " on " . $Identity . " is above threshold: " . \ ("The " . $Name . " on " . $Identity . " is above threshold: " . \
$CheckHealthCurrent->$Temperature . "\C2\B0" . "C"); $Temperature . "\C2\B0" . "C");
:set ($CheckHealthTemperatureNotified->$Temperature) true; :set ($CheckHealthTemperatureNotified->$Name) true;
} }
:if ($CheckHealthCurrent->$Temperature <= ($CheckHealthTemperature->$Temperature - $CheckHealthTemperatureDeviation) && \ :if ($Temperature <= ($CheckHealthTemperature->$Name - $CheckHealthTemperatureDeviation) && \
$CheckHealthTemperatureNotified->$Temperature = true) do={ $CheckHealthTemperatureNotified->$Name = true) do={
$SendNotification ([ $SymbolForNotification "white-heavy-check-mark" ] . "Health recovery: " . $Temperature) \ $SendNotification ([ $SymbolForNotification "white-heavy-check-mark" ] . "Health recovery: " . $Name) \
("The " . $Temperature . " on " . $Identity . " dropped below threshold: " . \ ("The " . $Name . " on " . $Identity . " dropped below threshold: " . \
$CheckHealthCurrent->$Temperature . "\C2\B0" . "C"); $Temperature . "\C2\B0" . "C");
:set ($CheckHealthTemperatureNotified->$Temperature) false; :set ($CheckHealthTemperatureNotified->$Name) false;
} }
} }
} }