Skip to content

Commit e0c687e

Browse files
authored
Remove extra logging in Growatt (home-assistant#157788)
1 parent 9823621 commit e0c687e

File tree

3 files changed

+1
-8
lines changed

3 files changed

+1
-8
lines changed

homeassistant/components/growatt_server/__init__.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,6 @@ def get_device_list_classic(
3737
login_response = api.login(config[CONF_USERNAME], config[CONF_PASSWORD])
3838
# DEBUG: Log the actual response structure
3939
except Exception as ex:
40-
_LOGGER.error("DEBUG - Login response: %s", login_response)
4140
raise ConfigEntryError(
4241
f"Error communicating with Growatt API during login: {ex}"
4342
) from ex

homeassistant/components/growatt_server/coordinator.py

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -113,9 +113,6 @@ def _sync_update_data(self) -> dict[str, Any]:
113113
min_settings = self.api.min_settings(self.device_id)
114114
min_energy = self.api.min_energy(self.device_id)
115115
except growattServer.GrowattV1ApiError as err:
116-
_LOGGER.error(
117-
"Error fetching min device data for %s: %s", self.device_id, err
118-
)
119116
raise UpdateFailed(f"Error fetching min device data: {err}") from err
120117

121118
min_info = {**min_details, **min_settings, **min_energy}
@@ -180,7 +177,6 @@ async def _async_update_data(self) -> dict[str, Any]:
180177
try:
181178
return await self.hass.async_add_executor_job(self._sync_update_data)
182179
except json.decoder.JSONDecodeError as err:
183-
_LOGGER.error("Unable to fetch data from Growatt server: %s", err)
184180
raise UpdateFailed(f"Error fetching data: {err}") from err
185181

186182
def get_currency(self):

homeassistant/components/growatt_server/quality_scale.yaml

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,9 +32,7 @@ rules:
3232
status: todo
3333
comment: Replace bare Exception catches in __init__.py with specific growattServer exceptions.
3434
integration-owner: done
35-
log-when-unavailable:
36-
status: todo
37-
comment: Remove _LOGGER.error calls in coordinator.py when raising UpdateFailed (choose one, not both).
35+
log-when-unavailable: done
3836
parallel-updates: done
3937
reauthentication-flow: todo
4038
test-coverage: todo

0 commit comments

Comments
 (0)