Always retain Home Assistant MQTT Discovery config topic (#777)

* Fix issue where MQTT Discovery fails on broker reconnection * Always retain the config topic/payload * Fix tests
bachya · Nov 22, 2023 · 863401d · 863401d
1 parent 2e04ac3
commit 863401d
Show file tree

Hide file tree

Showing 2 changed files with 444 additions and 442 deletions.
diff --git a/ecowitt2mqtt/helpers/publisher/mqtt/hass.py b/ecowitt2mqtt/helpers/publisher/mqtt/hass.py
@@ -572,7 +572,9 @@ async def async_publish(self, data: dict[str, CalculatedValueType]) -> None:
                                     },
                                 )
                             ),
-                            retain=self._config.mqtt_retain,
+                            # We always retain the config payload:
+                            # https://github.com/bachya/ecowitt2mqtt/issues/760#issuecomment-1821340217
+                            retain=True,
                         )
                     )
                 )