Home-Assistant
/
core
mirror of https://github.com/home-assistant/core.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Add cooldown timer before Sonos resubscriptions (#68521)

pull/68592/head
jjlawren 2022-03-23 16:20:20 -05:00 committed by Paulus Schoutsen
parent 9636435ff2
commit 454cb44ee8
1 changed files with 16 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
homeassistant/components/sonos/speaker.py
View File

@ -63,6 +63,7 @@ from .media import SonosMedia
from .statistics import ActivityStatistics, EventStatistics from .statistics import ActivityStatistics, EventStatistics
NEVER_TIME = -1200.0 NEVER_TIME = -1200.0
RESUB_COOLDOWN_SECONDS = 10.0
EVENT_CHARGING = { EVENT_CHARGING = {
"CHARGING": True, "CHARGING": True,
"NOT_CHARGING": False, "NOT_CHARGING": False,
@ -126,6 +127,7 @@ class SonosSpeaker:
self._last_event_cache: dict[str, Any] = {} self._last_event_cache: dict[str, Any] = {}
self.activity_stats: ActivityStatistics = ActivityStatistics(self.zone_name) self.activity_stats: ActivityStatistics = ActivityStatistics(self.zone_name)
self.event_stats: EventStatistics = EventStatistics(self.zone_name) self.event_stats: EventStatistics = EventStatistics(self.zone_name)
self._resub_cooldown_expires_at: float | None = None
# Scheduled callback handles # Scheduled callback handles
self._poll_timer: Callable | None = None self._poll_timer: Callable | None = None
@ -502,6 +504,16 @@ class SonosSpeaker:
@callback @callback
def speaker_activity(self, source): def speaker_activity(self, source):
"""Track the last activity on this speaker, set availability and resubscribe.""" """Track the last activity on this speaker, set availability and resubscribe."""
if self._resub_cooldown_expires_at:
if time.monotonic() < self._resub_cooldown_expires_at:
_LOGGER.debug(
"Activity on %s from %s while in cooldown, ignoring",
self.zone_name,
source,
)
return
self._resub_cooldown_expires_at = None
_LOGGER.debug("Activity on %s from %s", self.zone_name, source) _LOGGER.debug("Activity on %s from %s", self.zone_name, source)
self._last_activity = time.monotonic() self._last_activity = time.monotonic()
self.activity_stats.activity(source, self._last_activity) self.activity_stats.activity(source, self._last_activity)
@ -542,6 +554,10 @@ class SonosSpeaker:
if not self.available: if not self.available:
return return
if self._resub_cooldown_expires_at is None and not self.hass.is_stopping:
self._resub_cooldown_expires_at = time.monotonic() + RESUB_COOLDOWN_SECONDS
_LOGGER.debug("Starting resubscription cooldown for %s", self.zone_name)
self.available = False self.available = False
self.async_write_entity_states() self.async_write_entity_states()