Skip to content

Commit

Permalink
Lower "Activity on mDNS" message
Browse files Browse the repository at this point in the history
Users seem to get added mDNS events fairly regularly, and with that
the log message. However, the trigger_resubscribe_if_scheduled does not
do anything typically, so let's lower the log level here to avoid
polluting the log unnecessarily.
  • Loading branch information
agners committed Sep 12, 2024
1 parent f29cc35 commit d74782c
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion matter_server/server/device_controller.py
Original file line number Diff line number Diff line change
Expand Up @@ -1481,7 +1481,13 @@ def _on_mdns_operational_node_state(
self._setup_node_create_task(node_id)
elif state_change == ServiceStateChange.Added:
# Trigger node re-subscriptions when mDNS entry got added
node_logger.info("Activity on mDNS, trigger resubscribe")
# Note: Users seem to get such mDNS messages fairly regularly, and often
# the subscription to the device is healthy nad fine. This is not a problem
# since trigger_resubscribe_if_scheduled won't do anything in that case
# (no resubscribe is scheduled).
# But this does speedup the resubscription process in case the subscription
# is already in resubscribe mode.
node_logger.debug("Activity on mDNS, trigger resubscribe if scheduled")
asyncio.create_task(
self._chip_device_controller.trigger_resubscribe_if_scheduled(
node_id, "mDNS state change detected"
Expand Down

0 comments on commit d74782c

Please sign in to comment.