-
-
Notifications
You must be signed in to change notification settings - Fork 566
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
Tibber Pulse: fix server going silent #7575
Conversation
Das müsst ihr bitte ausprobieren- der Code ist nicht schön... |
Irgendwas passt da noch nicht. Ich habe gerade zwei Effekte festgestellt nachdem ich das Netzwerk getrennt und nach einigen Minuten wieder angeschaltet habe. Erstens: Doppelte Anzahl Reconnects aus der GraphQL Libary:
Hier liefen zwei pro Sekunde. Normalerweise probiert die Library sekündlich neu zu verbinden. Zweitens: Nach erfolgreichen Reconnect (ich sehe Pulse Antworten im Log - übrigens nicht doppelt), scheint die 10s Regelschleife nicht mehr zu laufen. Ich sehe bei mir noch Easee und Pulse Logeinträge aber nichts mehr aus der Site. |
Kann gerade nicht testen und habe auch nicht in den Code geschaut. Wenn der eigene Reconnect gestartet wird, muss aber auf alle Fälle der alte Connect vom Client beendet werden, soweit ich die API verstehe. |
Wenns vorher einen Timeout gab sollte das |
Ich hattest gestern wieder einen Hänger und musste evcc neu starten 😔 @andig magst du dir das Thema noch mal anschauen? |
Ich glaube "closed" für den PR ist doof, oder? War nicht der letzte Stand von @naltatis, dass doppelte Verbindungen auftraten? |
@foto-andreas ja, ich wollte den wieder öffnen, hab aber nur das Stale Tag gelöscht. |
Ich verstehe aus meiner Entwickler-Sicht das Konzept des Öffnens und Schließen von Tickets hier noch nicht. @andig bist du prinzipiell dagegen, hier etwas in evcc zu fixen? Ich würde erwarten, dass wir hier gemeinsam nach einer Lösung suchen und dann was funktionierendes in evcc einbauen. |
Hier wurde kein Ticket geschlossen sondern ein nicht funktionierender PR. Mangels Tibber kann ich das nicht weiter voran treiben.
Das ist meine absolute Lieblingsformulierung :) Wenn jemand eine Idee für einen funktionierenden Fix hat sehr gerne. |
Alles klar, das Statement ist für mich völlig ok. Dann schaue ich mir das die Tage mal an und übe mich etwas in go. |
PR kann getestet werden. |
Meine ersten Tests sehen gut aus. Reconnect bei unterbrochenem Netzwerk funktionieren weiterhin wie erwartet. Den Fall "Verbindung steht, aber es kommen keine Daten mehr" kann ich natürlich nicht einfach nachstellen. Ich lass das jetzt mal über Nacht laufen und meld mich. Im Zweifelsfall probieren wir das mit dem Fix erstmal. |
Ihr seid ja schneller als die Polizei erlaubt... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ist bei mir jetzt die Nacht durchgelaufen und hat alle erzwungenen Verbindungsabbrüche erfolgreich wiederhergestellt. Ich würde sagen Todos und auskommentierten Code rausnehmen und ab in den Master.
Super. Bei mir ist seit dem Merge alles problemlos durchgelaufen. Vielen Dank! |
Ja bei mir ebenfalls! |
Fix #7314