Skip to content

Commit 08337fa

Browse files
committed
Demote redis update log messages to debug with state transition context
The "will send" messages were at info level causing noise in the logs. Lower them to debug and include the state transition (old -> new) for more context alongside the outgoing messages. Also include the state transition in the "Appliance updated by redis" debug message. Assisted-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent d44c5cc commit 08337fa

File tree

1 file changed

+13
-7
lines changed

1 file changed

+13
-7
lines changed

home/process.py

Lines changed: 13 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,11 @@ async def _update_performers_by_protocol_trigger(self, scheduler, trigger):
9898
async def _on_appliance_updated_by_redis(self, scheduler, new_appliance):
9999
old_appliance = self._appliances[new_appliance.name]
100100
old_state, new_state = old_appliance.update(new_appliance)
101-
self._logger.debug("Appliance {} updated by redis".format(new_appliance.name))
101+
self._logger.debug(
102+
"Appliance {} updated by redis ({} -> {})".format(
103+
new_appliance.name, old_state.compute(), new_state.compute()
104+
)
105+
)
102106
for performer in [
103107
performer
104108
for performer in self._performers
@@ -113,9 +117,10 @@ async def _on_appliance_updated_by_redis(self, scheduler, new_appliance):
113117
)
114118
msgs = performer.execute(old_state, new_state)
115119
if msgs:
116-
self._logger.info(
117-
"Performer {} updated by redis will send {}".format(
118-
performer.name, msgs
120+
self._logger.debug(
121+
"Performer {} sending {} ({} -> {})".format(
122+
performer.name, msgs,
123+
old_state.compute(), new_state.compute()
119124
)
120125
)
121126
for writer in self._protocols_writers:
@@ -128,9 +133,10 @@ async def _on_performer_updated_by_redis(self, performer, old_state, new_state):
128133
msgs = performer.execute(old_state, new_state)
129134
self._logger.debug("Performer {} updated by redis".format(performer.name))
130135
if msgs:
131-
self._logger.info(
132-
"Performer {} updated by redis will send {}".format(
133-
performer.name, msgs
136+
self._logger.debug(
137+
"Performer {} sending {} ({} -> {})".format(
138+
performer.name, msgs,
139+
old_state.compute(), new_state.compute()
134140
)
135141
)
136142
for writer in self._protocols_writers:

0 commit comments

Comments
 (0)