Skip to content
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

MQTT Client does not reconnect #2510

Closed
martingruening opened this issue Feb 2, 2024 · 12 comments
Closed

MQTT Client does not reconnect #2510

martingruening opened this issue Feb 2, 2024 · 12 comments

Comments

@martingruening
Copy link
Contributor

Bug Report or Feature Request (mark with an x)

- [X] bug report -> please search issues before submitting
- [ ] feature request

Bug description or desired functionality.

MQTT Client does not reconnect after network connection

Setup: OpenEMS Edge 2024.1 in Docker container
Sending topics to an Mosquitto MQTT broker
Works for weeks without a hickup, but when a network interuption happens (e.g. rebooting the router after an update) the MQTT client does not reconnect to the broker

Docker Log:

2024-02-02T13:17:10,476 [Worker-0] INFO  [api.mqtt.ControllerApiMqttImpl] [ctrlControllerApiMqtt] Client is not connected
2024-02-02T13:17:10,476 [Worker-0] INFO  [i.mqtt.SendChannelValuesWorker] [ctrlControllerApiMqtt] Error while sending MQTT topics: scheduler0/_PropertyId [...]

The 'Client is not connected' message repeats a lot, but no connection is rebuilt.
Only fix is to restart the Docker container.

Expected behaviour would be to retry to rebuild the connection continously.

@Sn0w3y
Copy link
Contributor

Sn0w3y commented Feb 4, 2024

Only fix is to restart the Docker container.

Have you already tried to deactivate and activate the Controller Api MQTT ? Like this should work aswell :)

@Sn0w3y
Copy link
Contributor

Sn0w3y commented Feb 6, 2024

Hallo @martingruening,

kannst du mir deine Config für dein MQTT mal geben, damit ich meinen fix eventuell testen kann?

Wäre super, da ich mich zwar mit dem Java Code auskenne, aber leider nicht mit MQTT 👯‍♂️

@Sn0w3y
Copy link
Contributor

Sn0w3y commented Feb 6, 2024

@martingruening kannst dir mal meinen Approach ansehen und testen bitte - PullRequest

@martingruening
Copy link
Contributor Author

@Sn0w3y Natürlich kann ich testen. Wie komme ich an ein jar-File, das den neuen Code enthält?

@Sn0w3y
Copy link
Contributor

Sn0w3y commented Feb 7, 2024

@Sn0w3y Natürlich kann ich testen. Wie komme ich an ein jar-File, das den neuen Code enthält?

kannst dir ja mit meinem Code in dem PullRequest selber bauen :)

Hab nur die eine Datei geändert :) #2525

Andernfalls baue ich dir die .jar heute Abend

@martingruening

@martingruening
Copy link
Contributor Author

Hallo @Sn0w3y - es wäre toll, wenn du mir ein JAR senden könntest, das ich testen kann. Sorry für die späte Antwort.

@Sn0w3y
Copy link
Contributor

Sn0w3y commented Feb 12, 2024

https://www.file-upload.net/download-15268371/openems.jar.html

Versuch die mal und bitte berichte ! :) Sieh dir auch die Logs an, hab mal ein paar mehr Details hinzugefügt um zu Debuggen.

@martingruening

@Sn0w3y
Copy link
Contributor

Sn0w3y commented Feb 12, 2024

@martingruening hast du bereits testen können ? :)

@Sn0w3y
Copy link
Contributor

Sn0w3y commented Feb 23, 2024

@martingruening gibt es News ?

@martingruening
Copy link
Contributor Author

Entschuldigung. Die Emails von GitHub zu diesem Case sind leider im Spamordner gelandet. Der Change funktioniert - es wird jetzt ein Reconnect zum MQTT Broker durchgeführt. Danke!

@Sn0w3y
Copy link
Contributor

Sn0w3y commented Feb 28, 2024 via email

@sfeilmeier
Copy link
Contributor

Discussion is continued in the respective PRs (e.g. #2525) and/or on OpenEMS Community https://community.openems.io/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants