Fix ups checks with Python ValueError exceptions

Signed-off-by: Sven Rueß <[email protected]>

Author: sven-ruess

cmk/base/legacy_checks/ups_in_voltage.py

@@ -13,7 +13,10 @@
 
 
 def discover_ups_in_voltage(info):
-    yield from ((item, {}) for item, value in info if int(value) > 0)
+    try:
+        yield from ((item, {}) for item, value in info if int(value) > 0)
+    except ValueError:
+        pass
 
 
 def parse_ups_in_voltage(string_table: StringTable) -> StringTable:

cmk/base/legacy_checks/ups_out_voltage.py

@@ -15,7 +15,17 @@
 
 
 def discover_ups_out_voltage(info: list[list[str]]) -> Iterable[tuple[str, dict]]:
-    yield from ((item, {}) for item, value, *_rest in info if int(value) > 0)
+    for (
+        item,
+        value,
+    ) in info:
+        try:
+            value_int = int(value)
+        except ValueError:
+            value_int = 0
+
+        if value_int > 0:
+            yield (item, {})
 
 
 def parse_ups_out_voltage(string_table: StringTable) -> StringTable:

cmk/plugins/collection/agent_based/ups_out_load.py

@@ -34,7 +34,10 @@ class UpsPowerVoltage(NamedTuple):
 def int_or_zero(value: str) -> int:
     if value == "":
         return 0
-    return int(value)
+    try:
+        return int(value)
+    except ValueError:
+        return 0
 
 
 def parse_ups_load(string_table: Sequence[StringTable]) -> Section:

tests/unit/cmk/plugins/prometheus/server_side_calls/test_special_agent.py

@@ -93,7 +93,7 @@
                     "--cert-server-name",
                     "host",
                     "auth_token",
-                    "--token",
+                    "--token-reference",
                     Secret(0),
                 ],
             ),