From 51584f083951cbe0620e015ccde35ee103a20c68 Mon Sep 17 00:00:00 2001 From: Sahas Subramanian Date: Thu, 12 Dec 2024 17:00:52 +0100 Subject: [PATCH] Remove "Checking battery" log message It is not providing any useful information or details about its context, and it is too noisy. Signed-off-by: Sahas Subramanian --- RELEASE_NOTES.md | 2 ++ .../_power_distributing/_component_managers/_battery_manager.py | 1 - 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/RELEASE_NOTES.md b/RELEASE_NOTES.md index 8af471ead..5d3ef7f14 100644 --- a/RELEASE_NOTES.md +++ b/RELEASE_NOTES.md @@ -61,3 +61,5 @@ This release includes a new `ConfigManager` class to simplify managing the confi - Fix a bug in `BackgroundService` where it won't try to `self.cancel()` and `await self.wait()` if there are no internal tasks. This prevented to properly implement custom stop logic without having to redefine the `stop()` method too. - Fix a bug where if a string was passed to the `ConfigManagingActor` it would be interpreted as a sequence of 1 character strings. + +- Remove a confusing log message in the power distributing actor. diff --git a/src/frequenz/sdk/microgrid/_power_distributing/_component_managers/_battery_manager.py b/src/frequenz/sdk/microgrid/_power_distributing/_component_managers/_battery_manager.py index d676faa46..bb75b9bf1 100644 --- a/src/frequenz/sdk/microgrid/_power_distributing/_component_managers/_battery_manager.py +++ b/src/frequenz/sdk/microgrid/_power_distributing/_component_managers/_battery_manager.py @@ -400,7 +400,6 @@ def _check_request( return Error(request=request, msg="Empty battery IDs in the request") for battery in request.component_ids: - _logger.debug("Checking battery %d", battery) if battery not in self._battery_caches: msg = ( f"No battery {battery}, available batteries: "