-
-
Notifications
You must be signed in to change notification settings - Fork 251
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
Mosquitto update? #1543
Comments
I think we should wait for now, as OH probably has not updated the binding for MQTT for the updated version and as such it would be counterproductive to switch it now. I did not check to see if has been updated, it may have been but I would guess it has not. I would still keep this issue open though because once OH does update the binding it will be a good idea to do so. |
I'd think we should move to the mosquitto repo first. |
|
Why go against openHAB, we want to provide the optimal user experience for openHAB so doing this would be contrary to one of our founding goals. |
?? why is this against OH ? I just mean to upgrade to latest mosquitto (which is supposed to be downwards compatible, of course we would need to test) |
If the latest upgrade is not compatible with OH it doesn't make sense to upgrade it unless I'm misunderstanding. |
Of course. But I don't think it's incompatible, and I'm saying we need to test this before we merge. |
Ahh, ok then. That makes more sense. If you want to create an initial PR I'll be more than happy to help review and test the changes. I'm just a little wary of potentially breaking stuff that already works. As the old adage goes "Don't fix what ain't broke". |
The 2.0 release notes say it brings a number of breaking changes. So let's stay with 1.5.7+ for the time being. |
I saw that the mqtt-binding in openhab now can be configured to use version 5. |
None that I know of. Should you our anyone be willing to look into this, please provide an installation routine that also works unattended, providing a default mosquitto config. |
I would prefer to extend the current install routine. Maybe something like:
the rest would stay the same except a default password for unattended mode. |
I won't prescribe how to implement but we want everything in openHABian to work a) unattended and b) interactive. If unattended, there cannot exist any prior installation. On versions, we need separate menu items to select 1.x or 2.x and eventually a selector variable for unattended installations but staying with 1.x there should be fine for the time being. |
@Larsen-Locke any plan on this ? |
not really, because I was not sure how to do this without the risk of causing trouble: As far as I understood the recent debian versions don't support mosquitto 2, so you have to import keys and then start the installation that fits to the debian version (buster, stretch etc.). The current bullseye lite supports mosquitto 2 so the installation works directly. |
Makes sense. Let's keep watching for now. |
Any work in progress? |
well, not from my side. |
We have v1.5.7, this is in debian/RaspiOS main repo.
But MQTTv5 requires the new v2.0+
See https://mosquitto.org/blog/2013/01/mosquitto-debian-repository/.
Let's change to install from this dedicated repo.
We should also add a BATS test to avoid the following pitfall.
Attention: take care of breaking changes, latest mosquitto broke HA, see also
https://www.heise.de/news/Smarthome-mit-Home-Assistant-Vorsicht-bei-Update-des-MQTT-Brokers-6120728.html
The text was updated successfully, but these errors were encountered: