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

Homeassistant MQTT: neue Namenrichtlinie #1194

Closed
smoki3 opened this issue Aug 2, 2023 · 29 comments
Closed

Homeassistant MQTT: neue Namenrichtlinie #1194

smoki3 opened this issue Aug 2, 2023 · 29 comments
Labels
bug Something isn't working

Comments

@smoki3
Copy link

smoki3 commented Aug 2, 2023

What happened?

Hallo,

seit dem aktuellen Update von Home Assistant bekommt man die Meldung, dass die übertragenen MQTT Namen nicht mehr zulässig sind und aktuell noch ein Workaround betrieben wird.

image

To Reproduce Bug

Update Homeassistant auf 2023.08

Expected Behavior

kompatibilität mit Homeassistant

Install Method

Pre-Compiled binary from GitHub

What git-hash/version of OpenDTU?

v23.8.1

Relevant log/trace output

No response

Anything else?

No response

@smoki3 smoki3 added the bug Something isn't working label Aug 2, 2023
@knurrdog
Copy link

knurrdog commented Aug 2, 2023

The naming of MQTT entities changes to correspond with HA guidelines/

@LennartF22
Copy link
Contributor

Sollte kein großes Problem sein, die Namen in der Auto-Discovery-Config anzupassen. Falls noch keiner dran ist, könnte ich mich darum kümmern und eine PR stellen (dann am besten wahrscheinlich erstmal nur als Opt-in bei den MQTT-Einstellungen, damit es bei alten Home Assistant Versionen nichts durcheinander bringt).

@heilingbrunner
Copy link

Könnt Ihr das am Ende etwas detailierter erklären (welches Gerät, welches Menü, etc.), damit man das nachvollziehen kann ?
Laut HA soll man diese Naming-Convention Warnung ignorieren und alles geht wie vorher, aber bei mir sind alle Werte 'unavailable'.
Danke.

@Norm71
Copy link

Norm71 commented Aug 5, 2023

Wenn der Wechselrichter abschaltet sind die Werte nicht verfügbar. Sobald er produziert sollten die Werte wieder zu sehen sein. So ist es zumindest bei mir

@heilingbrunner
Copy link

Ja, dass nachts keine Werte kommen ist bekannt. Aber ich sehe im openDTU Web-UI, dass das Gerät funktioniert. Hat ja auch wochenlang funktioniert in Home-Assistant. Der 'binary_sensor.hm800_producing' ist hingegen immer noch verfügbar. Alle 'sensor.hm800_...' Entities nicht. Die wurden in der Warnung ja auch moniert. Hm ...

@LennartF22
Copy link
Contributor

Habe in den MQTT-Einstellungen eine Option eingebaut, um auf das neue Namensschema zu wechseln. Falls es jemand ausprobieren möchte: https://github.com/LennartF22/OpenDTU/actions/runs/5773100117

Kann natürlich nicht sagen, ob und wann das in dieser Form gemerged wird.

@leowinterde
Copy link

Rel. lumapu/ahoy#1066

@sholvatealc
Copy link

Habe in den MQTT-Einstellungen eine Option eingebaut, um auf das neue Namensschema zu wechseln. Falls es jemand ausprobieren möchte: https://github.com/LennartF22/OpenDTU/actions/runs/5773100117

Kann natürlich nicht sagen, ob und wann das in dieser Form gemerged wird.

Hab des jetzt mal angewendet. Muss ich im Homeassistant dann auch noch eine Aktion ausführen?

@sarmbruster
Copy link

Vielen Dank @LennartF22 . Ich habe es bei mir installiert und in der UI Altes Namensschema abgeschaltet. In HA ( 2023.8.1) ist die oben beschriebene Fehlermeldung weg.
Daher strong +1 fürs mergen.

@wesseli01
Copy link

Vielen Dank @LennartF22. Ebenso in der UI das alte Namensschema deaktiviert. Alle Entitaeten sind nun sauber angelegt worden. Die Funktion sollte im neuen Release integriert werden.

@morph027
Copy link

Auch von mir ein 👍 , läuft.

@Omega13x
Copy link

Hallo.
Ich habe die Version auch mal installiert und die alten Namensrichtlinien deaktiviert. Ich habe OpenDTU und HA komplett neu gestartet. Ich kann aber keine Änderung feststellen. Habe ich irgendetwas vergessen?

@wesseli01
Copy link

@Omega13x und was sehen Sie unter den Integrationseinträgen von MQTT?

@wesseli01
Copy link

@LennartF22 - Könnten Sie die Änderung in die neue Version 23.8.8 integrieren?

@LennartF22
Copy link
Contributor

@wesseli01 Erledigt, sollte gleich hier verfügbar sein: https://github.com/LennartF22/OpenDTU/actions/runs/5836940847

@wesseli01
Copy link

@LennartF22 , Vielen vielen Dank. Das hilft uns sehr.

@Phil1988
Copy link

Wie ist der aktuelle Stand? Wird das in die offizielle Version hier integriert?
Ich habe es nicht von Lennarts Version getestet, weil ich hier die originale Version nehmen möchte und später dann das FW Update durchführen möchte, sobald es integriert ist.
Danke @LennartF22 übrigens!

@Gitsarry
Copy link

The OP warning disappeared for me after update to HA 2023.8.4.
Matches this point of 2023.8.4 changelog:

@MarpleA
Copy link

MarpleA commented Aug 30, 2023

ich bin auf Version 23.8.28.
Wo finde ich den die Option zum Umstellen und wie heißt sie genau?

@sholvatealc
Copy link

ich bin auf Version 23.8.28. Wo finde ich den die Option zum Umstellen und wie heißt sie genau?

Die ist in einer ich sag jetzt mal "privaten" Änderung von @LennartF22 hier https://github.com/LennartF22/OpenDTU/actions/runs/5836940847

Dann bei Einstellung/MQTT ganz unten.

@MarpleA
Copy link

MarpleA commented Aug 31, 2023

ah ok. danke. ich dachte das wäre schon in das offizielle Release eingefloßen.

@Heiko-L
Copy link

Heiko-L commented Sep 10, 2023

The OP warning disappeared for me after update to HA 2023.8.4. Matches this point of 2023.8.4 changelog:

* Remove repair issue for MQTT discovered items [Remove repair issue for MQTT discovered items home-assistant/core#98768](https://github.com/home-assistant/core/pull/98768)

That is no solutuion to the problem

@sholvatealc
Copy link

sholvatealc commented Sep 10, 2023

@Heiko-L
In the meantime, use this version until it is incorporated into the official one.
Then uncheck “Old naming scheme” at the bottom under Settings/MQTT.
https://github.com/LennartF22/OpenDTU/actions/runs/5836940847

I use HA 2023.9.1

@sholvatealc
Copy link

@LennartF22
Magst Du das aktuelle Built mit Deinem Namenschalter reinstellen ? Oder ist des jetzt schon abgehandelt das Thema?
Grüße Markus

@LennartF22
Copy link
Contributor

LennartF22 commented Oct 5, 2023

@sholvatealc Ich könnte meine Änderungen zwar nochmal mit der aktuellen OpenDTU-Version zusammenführen, aber das ist ja irgendwie nicht Sinn der Sache. Stattdessen müsste entweder meine PR von @tbnobody gemerged werden oder das Problem anderweitig angegangen werden. Nur weil die Meldung nicht mehr ganz so „in your Face“ ist, besteht das unterliegende Problem ja weiterhin. Spätestens wenn Home Assistant den Legacy-Mode in ein paar Monaten entfernt, muss ja irgendetwas passieren (wenn bei den Leuten nicht überall der Wechselrichtername doppelt in den Entitäten drin stehen soll).

@S-Przybylski
Copy link

Dear @tbnobody could you have a look on this please?
Each restart of Home Assistant my protocol is flodded with warnings. Thanks

@TheDK
Copy link

TheDK commented Nov 14, 2023

+1 - this should be changed as it is now HA standard and will hit every new user...

@tbnobody
Copy link
Owner

Implemented in 71d1b3b

Copy link

github-actions bot commented Apr 2, 2024

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new discussion or issue for related concerns.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests