Skip to content

Commit fdc452c

Browse files
committed
Broaden exception handling when setting power through microgrid API
Signed-off-by: Sahas Subramanian <[email protected]>
1 parent 987e411 commit fdc452c

File tree

3 files changed

+20
-0
lines changed

3 files changed

+20
-0
lines changed

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

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -672,6 +672,14 @@ def _parse_result(
672672
battery_ids,
673673
request_timeout.total_seconds(),
674674
)
675+
except Exception as e:
676+
failed_power += distribution[inverter_id]
677+
failed_batteries = failed_batteries.union(battery_ids)
678+
_logger.warning(
679+
"Unknown error while setting power to battery %s: %s",
680+
battery_ids,
681+
e,
682+
)
675683

676684
return failed_power, failed_batteries
677685

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

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -351,6 +351,12 @@ async def _set_api_power(
351351
component_id,
352352
err,
353353
)
354+
case e if e is not None:
355+
_logger.warning(
356+
"Unknown error while setting power to EV charger %s: %s",
357+
component_id,
358+
e,
359+
)
354360
if failed_components:
355361
return PartialFailure(
356362
failed_components=failed_components,

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

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -201,6 +201,12 @@ async def _set_api_power(
201201
component_id,
202202
err,
203203
)
204+
case e if e is not None:
205+
_logger.warning(
206+
"Unknown error while setting power to EV charger %s: %s",
207+
component_id,
208+
e,
209+
)
204210
if failed_components:
205211
await self._results_sender.send(
206212
PartialFailure(

0 commit comments

Comments
 (0)