refactor(assets): Removed the major issues parameter from the operational lifetime extraction function.

Signed-off-by: eduardiazf <[email protected]>

Author: eduardiazf

src/frequenz/client/assets/electrical_component/_connection_proto.py

@@ -73,9 +73,7 @@ def component_connection_from_proto_with_issues(
         )
         return None
 
-    lifetime = _get_operational_lifetime_from_proto(
-        message, major_issues=major_issues, minor_issues=minor_issues
-    )
+    lifetime = _get_operational_lifetime_from_proto(message, minor_issues=minor_issues)
 
     return ComponentConnection(
         source=source_component_id,
@@ -87,18 +85,11 @@ def component_connection_from_proto_with_issues(
 def _get_operational_lifetime_from_proto(
     message: electrical_components_pb2.ElectricalComponentConnection,
     *,
-    major_issues: list[str],
     minor_issues: list[str],
 ) -> Lifetime:
     """Get the operational lifetime from a protobuf message."""
     if message.HasField("operational_lifetime"):
-        try:
-            return lifetime_from_proto(message.operational_lifetime)
-        except ValueError as exc:
-            major_issues.append(
-                f"invalid operational lifetime ({exc}), considering it as missing "
-                "(i.e. always operational)",
-            )
+        return lifetime_from_proto(message.operational_lifetime)
     else:
         minor_issues.append(
             "missing operational lifetime, considering it always operational",