mirror of
https://github.com/home-assistant/core.git
synced 2025-07-17 02:07:09 +00:00
improve debug logging (#47858)
This commit is contained in:
Michael
GitHub
parent
8b3dccb1b4
commit
07c197687f
@ -191,7 +191,9 @@ async def async_setup_entry(hass: HomeAssistantType, entry: ConfigEntry):
|
||||
try:
|
||||
await api.async_setup()
|
||||
except (SynologyDSMLoginFailedException, SynologyDSMRequestException) as err:
|
||||
_LOGGER.debug("Unable to connect to DSM during setup: %s", err)
|
||||
_LOGGER.debug(
|
||||
"Unable to connect to DSM '%s' during setup: %s", entry.unique_id, err
|
||||
)
|
||||
raise ConfigEntryNotReady from err
|
||||
|
||||
hass.data.setdefault(DOMAIN, {})
|
||||
@ -401,7 +403,8 @@ class SynoApi:
|
||||
self.dsm.apis.get(SynoSurveillanceStation.CAMERA_API_KEY)
|
||||
)
|
||||
_LOGGER.debug(
|
||||
"State of Surveillance_station during setup:%s",
|
||||
"State of Surveillance_station during setup of '%s': %s",
|
||||
self._entry.unique_id,
|
||||
self._with_surveillance_station,
|
||||
)
|
||||
|
||||
@ -413,9 +416,7 @@ class SynoApi:
|
||||
@callback
|
||||
def subscribe(self, api_key, unique_id):
|
||||
"""Subscribe an entity to API fetches."""
|
||||
_LOGGER.debug(
|
||||
"Subscribe new entity - api_key:%s, unique_id:%s", api_key, unique_id
|
||||
)
|
||||
_LOGGER.debug("Subscribe new entity: %s", unique_id)
|
||||
if api_key not in self._fetching_entities:
|
||||
self._fetching_entities[api_key] = set()
|
||||
self._fetching_entities[api_key].add(unique_id)
|
||||
@ -423,9 +424,7 @@ class SynoApi:
|
||||
@callback
|
||||
def unsubscribe() -> None:
|
||||
"""Unsubscribe an entity from API fetches (when disable)."""
|
||||
_LOGGER.debug(
|
||||
"Unsubscribe new entity - api_key:%s, unique_id:%s", api_key, unique_id
|
||||
)
|
||||
_LOGGER.debug("Unsubscribe entity: %s", unique_id)
|
||||
self._fetching_entities[api_key].remove(unique_id)
|
||||
if len(self._fetching_entities[api_key]) == 0:
|
||||
self._fetching_entities.pop(api_key)
|
||||
@ -437,7 +436,9 @@ class SynoApi:
|
||||
"""Determine if we should fetch each API, if one entity needs it."""
|
||||
# Entities not added yet, fetch all
|
||||
if not self._fetching_entities:
|
||||
_LOGGER.debug("Entities not added yet, fetch all")
|
||||
_LOGGER.debug(
|
||||
"Entities not added yet, fetch all for '%s'", self._entry.unique_id
|
||||
)
|
||||
return
|
||||
|
||||
# Determine if we should fetch an API
|
||||
@ -460,32 +461,47 @@ class SynoApi:
|
||||
|
||||
# Reset not used API, information is not reset since it's used in device_info
|
||||
if not self._with_security:
|
||||
_LOGGER.debug("Disable security api from being updated")
|
||||
_LOGGER.debug(
|
||||
"Disable security api from being updated for '%s'",
|
||||
self._entry.unique_id,
|
||||
)
|
||||
self.dsm.reset(self.security)
|
||||
self.security = None
|
||||
|
||||
if not self._with_storage:
|
||||
_LOGGER.debug("Disable storage api from being updated")
|
||||
_LOGGER.debug(
|
||||
"Disable storage api from being updatedf or '%s'", self._entry.unique_id
|
||||
)
|
||||
self.dsm.reset(self.storage)
|
||||
self.storage = None
|
||||
|
||||
if not self._with_system:
|
||||
_LOGGER.debug("Disable system api from being updated")
|
||||
_LOGGER.debug(
|
||||
"Disable system api from being updated for '%s'", self._entry.unique_id
|
||||
)
|
||||
self.dsm.reset(self.system)
|
||||
self.system = None
|
||||
|
||||
if not self._with_upgrade:
|
||||
_LOGGER.debug("Disable upgrade api from being updated")
|
||||
_LOGGER.debug(
|
||||
"Disable upgrade api from being updated for '%s'", self._entry.unique_id
|
||||
)
|
||||
self.dsm.reset(self.upgrade)
|
||||
self.upgrade = None
|
||||
|
||||
if not self._with_utilisation:
|
||||
_LOGGER.debug("Disable utilisation api from being updated")
|
||||
_LOGGER.debug(
|
||||
"Disable utilisation api from being updated for '%s'",
|
||||
self._entry.unique_id,
|
||||
)
|
||||
self.dsm.reset(self.utilisation)
|
||||
self.utilisation = None
|
||||
|
||||
if not self._with_surveillance_station:
|
||||
_LOGGER.debug("Disable surveillance_station api from being updated")
|
||||
_LOGGER.debug(
|
||||
"Disable surveillance_station api from being updated for '%s'",
|
||||
self._entry.unique_id,
|
||||
)
|
||||
self.dsm.reset(self.surveillance_station)
|
||||
self.surveillance_station = None
|
||||
|
||||
@ -496,27 +512,32 @@ class SynoApi:
|
||||
self.network.update()
|
||||
|
||||
if self._with_security:
|
||||
_LOGGER.debug("Enable security api for updates")
|
||||
_LOGGER.debug("Enable security api updates for '%s'", self._entry.unique_id)
|
||||
self.security = self.dsm.security
|
||||
|
||||
if self._with_storage:
|
||||
_LOGGER.debug("Enable storage api for updates")
|
||||
_LOGGER.debug("Enable storage api updates for '%s'", self._entry.unique_id)
|
||||
self.storage = self.dsm.storage
|
||||
|
||||
if self._with_upgrade:
|
||||
_LOGGER.debug("Enable upgrade api for updates")
|
||||
_LOGGER.debug("Enable upgrade api updates for '%s'", self._entry.unique_id)
|
||||
self.upgrade = self.dsm.upgrade
|
||||
|
||||
if self._with_system:
|
||||
_LOGGER.debug("Enable system api for updates")
|
||||
_LOGGER.debug("Enable system api updates for '%s'", self._entry.unique_id)
|
||||
self.system = self.dsm.system
|
||||
|
||||
if self._with_utilisation:
|
||||
_LOGGER.debug("Enable utilisation api for updates")
|
||||
_LOGGER.debug(
|
||||
"Enable utilisation api updates for '%s'", self._entry.unique_id
|
||||
)
|
||||
self.utilisation = self.dsm.utilisation
|
||||
|
||||
if self._with_surveillance_station:
|
||||
_LOGGER.debug("Enable surveillance_station api for updates")
|
||||
_LOGGER.debug(
|
||||
"Enable surveillance_station api updates for '%s'",
|
||||
self._entry.unique_id,
|
||||
)
|
||||
self.surveillance_station = self.dsm.surveillance_station
|
||||
|
||||
async def async_reboot(self):
|
||||
@ -524,7 +545,10 @@ class SynoApi:
|
||||
try:
|
||||
await self._hass.async_add_executor_job(self.system.reboot)
|
||||
except (SynologyDSMLoginFailedException, SynologyDSMRequestException) as err:
|
||||
_LOGGER.error("Reboot not possible, please try again later")
|
||||
_LOGGER.error(
|
||||
"Reboot of '%s' not possible, please try again later",
|
||||
self._entry.unique_id,
|
||||
)
|
||||
_LOGGER.debug("Exception:%s", err)
|
||||
|
||||
async def async_shutdown(self):
|
||||
@ -532,7 +556,10 @@ class SynoApi:
|
||||
try:
|
||||
await self._hass.async_add_executor_job(self.system.shutdown)
|
||||
except (SynologyDSMLoginFailedException, SynologyDSMRequestException) as err:
|
||||
_LOGGER.error("Shutdown not possible, please try again later")
|
||||
_LOGGER.error(
|
||||
"Shutdown of '%s' not possible, please try again later",
|
||||
self._entry.unique_id,
|
||||
)
|
||||
_LOGGER.debug("Exception:%s", err)
|
||||
|
||||
async def async_unload(self):
|
||||
@ -540,11 +567,13 @@ class SynoApi:
|
||||
try:
|
||||
await self._hass.async_add_executor_job(self.dsm.logout)
|
||||
except (SynologyDSMAPIErrorException, SynologyDSMRequestException) as err:
|
||||
_LOGGER.debug("Logout not possible:%s", err)
|
||||
_LOGGER.debug(
|
||||
"Logout from '%s' not possible:%s", self._entry.unique_id, err
|
||||
)
|
||||
|
||||
async def async_update(self, now=None):
|
||||
"""Update function for updating API information."""
|
||||
_LOGGER.debug("Start data update")
|
||||
_LOGGER.debug("Start data update for '%s'", self._entry.unique_id)
|
||||
self._async_setup_api_requests()
|
||||
try:
|
||||
await self._hass.async_add_executor_job(
|
||||
@ -554,7 +583,11 @@ class SynoApi:
|
||||
_LOGGER.warning(
|
||||
"Connection error during update, fallback by reloading the entry"
|
||||
)
|
||||
_LOGGER.debug("Connection error during update with exception: %s", err)
|
||||
_LOGGER.debug(
|
||||
"Connection error during update of '%s' with exception: %s",
|
||||
self._entry.unique_id,
|
||||
err,
|
||||
)
|
||||
await self._hass.config_entries.async_reload(self._entry.entry_id)
|
||||
return
|
||||
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user