-
Notifications
You must be signed in to change notification settings - Fork 678
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve MQTT handling #971
Conversation
@jomjol, @f-fritz, @Varkzor, @pos-ei-don and @s0170071: Maybe you could test it as well with the attached |
Quite cool !
|
I deployed it now also to my live systems. Sadly there still seems to be an issue. it looks as if it only works every 2nd time:
I will have to have another look sometimes. |
See #953 (comment) Seems that with this branch, even if MQTT says "all good", you have to reboot with MQTT disabled, then enable again to have it working. |
I tried to improve it further, see #980 and the comments to it! |
This MR makes sure that it tries to establish a connection on every
ClassFlowMQTT
, if MQTT is enabled and it the connection is not established.Also it adds some log messages for easier debugging.
Below my test log:
Broker running on start
broker stopped during runtime
broker gets started again during runtime
Broker stopped on start
broker gets started during runtime
broker gets stopped again during runtime
MQTT disabled