Fix exception handling bugs in EV and PV managers

Also use `logging.exception` to log broad exceptions

Signed-off-by: Sahas Subramanian <[email protected]>

Author: shsms

src/frequenz/sdk/actor/power_distributing/_component_managers/_battery_manager.py

@@ -672,13 +672,12 @@ def _parse_result(
                     battery_ids,
                     request_timeout.total_seconds(),
                 )
-            except Exception as e:  # pylint: disable=broad-except
+            except Exception:  # pylint: disable=broad-except
                 failed_power += distribution[inverter_id]
                 failed_batteries = failed_batteries.union(battery_ids)
-                _logger.warning(
-                    "Unknown error while setting power to battery %s: %s",
+                _logger.exception(
+                    "Unknown error while setting power to batteries: %s",
                     battery_ids,
-                    e,
                 )
 
         return failed_power, failed_batteries

src/frequenz/sdk/actor/power_distributing/_component_managers/_ev_charger_manager/_ev_charger_manager.py

@@ -331,21 +331,20 @@ async def _set_api_power(
                 succeeded_components.add(component_id)
 
             match task.exception():
-                case asyncio.CancelledError:
+                case asyncio.CancelledError():
                     _logger.warning(
                         "Timeout while setting power to EV charger %s", component_id
                     )
-                case grpc.aio.AioRpcError as err:
+                case grpc.aio.AioRpcError() as err:
                     _logger.warning(
                         "Error while setting power to EV charger %s: %s",
                         component_id,
                         err,
                     )
-                case e if e is not None:
-                    _logger.warning(
-                        "Unknown error while setting power to EV charger %s: %s",
+                case Exception():
+                    _logger.exception(
+                        "Unknown error while setting power to EV charger: %s",
                         component_id,
-                        e,
                     )
         if failed_components:
             return PartialFailure(

src/frequenz/sdk/actor/power_distributing/_component_managers/_pv_inverter_manager/_pv_inverter_manager.py

@@ -191,21 +191,20 @@ async def _set_api_power(  # pylint: disable=too-many-locals
                 succeeded_components.add(component_id)
 
             match task.exception():
-                case asyncio.CancelledError:
+                case asyncio.CancelledError():
                     _logger.warning(
-                        "Timeout while setting power to EV charger %s", component_id
+                        "Timeout while setting power to PV inverter %s", component_id
                     )
-                case grpc.aio.AioRpcError as err:
+                case grpc.aio.AioRpcError() as err:
                     _logger.warning(
-                        "Error while setting power to EV charger %s: %s",
+                        "Error while setting power to PV inverter %s: %s",
                         component_id,
                         err,
                     )
-                case e if e is not None:
-                    _logger.warning(
-                        "Unknown error while setting power to EV charger %s: %s",
+                case Exception():
+                    _logger.exception(
+                        "Unknown error while setting power to PV inverter: %s",
                         component_id,
-                        e,
                     )
         if failed_components:
             await self._results_sender.send(