[Bug] MQTT looses connection

Nikey commented 10 months ago

Describe the bug MQTT loose connection after a few hours with mosquitto in Homeassistant. (tested in development branch)

To Reproduce Steps to reproduce the behavior:

Add MQTT connection to Homeassistant
Grab sensor data
Sensor data is grabbed but looses the connection for some reason after some hours

Expected behavior Sensor data is grabbed constantly.

Screenshots grafik

Additional context The MqttApiTopicHandler.cpp seems to send data after it looses connection (see log below), but HA don't recognize it for some reason.

A restart of mosquitto resolve the problem for another few hours. Unfortunatly don't have a log output from it..

BlueAndi commented 10 months ago

Which device really lost the connection,

Pixelix to mosquitto broker or
HA to mosquitto broker or
all to mosquitto broker?

Its not 100% clear for me.

The MqttApiTopicHandler.cpp seems to send data after it looses connection (see log below), but HA don't recognize it for some reason.

The log is incomplete here, it just shows the publishing of sensor data. Hmm ...

If there was a re-connection, Pixelix will publish all HA discovery infos and all sensor/plugin state information quite fast. Is there any limitation regarding message/ms which can be handled by mosquitto or HA? I can relax the publishing in case of a connection re-establishment.

Nikey commented 9 months ago

It unfortunatly didn't happen again. Did the update now, will reopen this issue with more information when the issue occured again. Thanks for your fast fixes :)

BlueAndi commented 9 months ago

It unfortunatly didn't happen again. Did the update now, will reopen this issue with more information when the issue occured again. Thanks for your fast fixes :)

Ok, you are welcome.

Nikey commented 9 months ago

It seems like it sometimes read false data from sensors ("nan" instead of the value of sensor, see logs below). The error happened on version 7.2.0

Logger: homeassistant.components.mqtt.models Source: components/mqtt/models.py:305 Integration: MQTT (documentation, issues) First occurred: 1. Januar 2024 um 15:14:25 (3 occurrences) Last logged: 10:21:24
Exception raised when updating state of sensor.sensors_0, topic: 'pixelix/sensors/0/temperature/state' with payload: b'{"value":"NAN"}'
Exception raised when updating state of sensor.sensors_1, topic: 'pixelix/sensors/1/humidity/state' with payload: b'{"value":"NAN"}'
Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/components/mqtt/models.py", line 305, in process_write_state_requests entity.async_write_ha_state() File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 745, in async_write_ha_state self._async_write_ha_state() File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 845, in _async_write_ha_state state, attr = self._async_generate_attributes() ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 786, in _async_generate_attributes state = self._stringify_state(available) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 751, in _stringify_state if (state := self.state) is None: ^^^^^^^^^^ File "/usr/src/homeassistant/homeassistant/components/sensor/init.py", line 634, in state raise ValueError( ValueError: Sensor sensor.sensors_0 has device class 'temperature', state class 'None' unit '°C' and suggested precision 'None' thus indicating it has a numeric value; however, it has the non-finite value: 'nan'

BlueAndi commented 8 months ago

@Nikey Can you open a separate issue for this, because its a different topic. Thx.

Nikey commented 8 months ago

Saw you already created one, will close this again.

BlueAndi / Pixelix

[Bug] MQTT looses connection #152