From c393622b64d2682718aac87d420f8208ea6dc1da Mon Sep 17 00:00:00 2001 From: "J. Nick Koston" Date: Sun, 17 Apr 2022 19:40:44 -1000 Subject: [PATCH] Avoid calling time.monotonic on coordinator refresh unless we are debugging (#70209) --- homeassistant/helpers/update_coordinator.py | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/homeassistant/helpers/update_coordinator.py b/homeassistant/helpers/update_coordinator.py index 0c0d647d48a..f7ad8e013cb 100644 --- a/homeassistant/helpers/update_coordinator.py +++ b/homeassistant/helpers/update_coordinator.py @@ -183,7 +183,8 @@ class DataUpdateCoordinator(Generic[_T]): if scheduled and self.hass.is_stopping: return - start = monotonic() + if log_timing := self.logger.isEnabledFor(logging.DEBUG): + start = monotonic() auth_failed = False try: @@ -255,12 +256,13 @@ class DataUpdateCoordinator(Generic[_T]): self.logger.info("Fetching %s data recovered", self.name) finally: - self.logger.debug( - "Finished fetching %s data in %.3f seconds (success: %s)", - self.name, - monotonic() - start, - self.last_update_success, - ) + if log_timing: + self.logger.debug( + "Finished fetching %s data in %.3f seconds (success: %s)", + self.name, + monotonic() - start, + self.last_update_success, + ) if not auth_failed and self._listeners and not self.hass.is_stopping: self._schedule_refresh()