From e68cd339b962b2f43064ed32e0eb37a78da56d58 Mon Sep 17 00:00:00 2001 From: Shawn Wilsher <656602+sdwilsh@users.noreply.github.com> Date: Tue, 31 Dec 2019 05:56:23 -0800 Subject: [PATCH] Reduce solaredge logging severity (#30305) * [solaredge] Reduce Severity of Log Line This log error happens frequently for some sites, but it shouldn't be an error. It is expected, per the SolarEdge Monitoring API, that some sites do not support this information, and the expected result is that this would be empty (see comments on #27959). Fixes #27959 * Fix a typo Co-authored-by: Fabian Affolter --- homeassistant/components/solaredge/sensor.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/homeassistant/components/solaredge/sensor.py b/homeassistant/components/solaredge/sensor.py index f0f1660a821..60cabaf38f0 100644 --- a/homeassistant/components/solaredge/sensor.py +++ b/homeassistant/components/solaredge/sensor.py @@ -41,7 +41,7 @@ async def async_setup_entry(hass, entry, async_add_entities): return _LOGGER.debug("Credentials correct and site is active") except KeyError: - _LOGGER.error("Missing details data in solaredge response") + _LOGGER.error("Missing details data in SolarEdge response") return except (ConnectTimeout, HTTPError): _LOGGER.error("Could not retrieve details from SolarEdge API") @@ -350,7 +350,9 @@ class SolarEdgePowerFlowDataService(SolarEdgeDataService): power_to = [] if "connections" not in power_flow: - _LOGGER.error("Missing connections in power flow data") + _LOGGER.debug( + "Missing connections in power flow data. Assuming site does not have any" + ) return for connection in power_flow["connections"]: