Test Netatmo 2.0.0 Alpha (GitHub)

[JohGre]

@apollon77 Hatte eben den ersten Absturz des Adapters, läuft aber weiter. Soll ich den Loglevel auf Info stellen, derzeit bin ich nur im warn unterwegs.

2022-09-08 14:39:22.960 - error: netatmo.0 (19841) Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch().
2022-09-08 14:39:22.960 - error: netatmo.0 (19841) unhandled promise rejection: homesdata error: No response
2022-09-08 14:39:22.962 - error: netatmo.0 (19841) Error: homesdata error: No response
at netatmo.handleRequestError (/opt/iobroker/node_modules/iobroker.netatmo/lib/netatmoLib.js:98:19)
at Request._callback (/opt/iobroker/node_modules/iobroker.netatmo/lib/netatmoLib.js:1126:25)
at self.callback (/opt/iobroker/node_modules/request/request.js:185:22)
at Request.emit (node:events:513:28)
at Request.emit (node:domain:489:12)
at Request.onRequestError (/opt/iobroker/node_modules/request/request.js:877:8)
at ClientRequest.emit (node:events:525:35)
at ClientRequest.emit (node:domain:489:12)
at TLSSocket.socketErrorListener (node:_http_client:481:9)
at TLSSocket.emit (node:events:513:28)
2022-09-08 14:39:22.963 - error: netatmo.0 (19841) homesdata error: No response
2022-09-08 14:39:22.974 - warn: netatmo.0 (19841) Terminated (UNCAUGHT_EXCEPTION): Without reason
2022-09-08 14:39:33.072 - error: host.VM-Slave Caught by controller[0]: This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). The promise rejected with the reason:
2022-09-08 14:39:33.075 - error: host.VM-Slave Caught by controller[1]: Error: homesdata error: No response
2022-09-08 14:39:33.075 - error: host.VM-Slave Caught by controller[1]: at netatmo.handleRequestError (/opt/iobroker/node_modules/iobroker.netatmo/lib/netatmoLib.js:98:19)
2022-09-08 14:39:33.075 - error: host.VM-Slave Caught by controller[1]: at Request._callback (/opt/iobroker/node_modules/iobroker.netatmo/lib/netatmoLib.js:1126:25)
2022-09-08 14:39:33.075 - error: host.VM-Slave Caught by controller[1]: at self.callback (/opt/iobroker/node_modules/request/request.js:185:22)
2022-09-08 14:39:33.076 - error: host.VM-Slave Caught by controller[1]: at Request.emit (node:events:513:28)
2022-09-08 14:39:33.076 - error: host.VM-Slave Caught by controller[1]: at Request.emit (node:domain:489:12)
2022-09-08 14:39:33.076 - error: host.VM-Slave Caught by controller[1]: at Request.onRequestError (/opt/iobroker/node_modules/request/request.js:877:8)
2022-09-08 14:39:33.076 - error: host.VM-Slave Caught by controller[1]: at ClientRequest.emit (node:events:525:35)
2022-09-08 14:39:33.076 - error: host.VM-Slave Caught by controller[1]: at ClientRequest.emit (node:domain:489:12)
2022-09-08 14:39:33.077 - error: host.VM-Slave Caught by controller[1]: at TLSSocket.socketErrorListener (node:_http_client:481:9)
2022-09-08 14:39:33.077 - error: host.VM-Slave Caught by controller[1]: at TLSSocket.emit (node:events:513:28)
2022-09-08 14:39:33.077 - error: host.VM-Slave instance system.adapter.netatmo.0 terminated with code 6 (UNCAUGHT_EXCEPTION)

[Shadowhunter23]

Installiert, soweit keine Probleme.
Innen-, Außen- und Regensensor

[oFbEQnpoLKKl6mbY5e13]

@apollon77

Ich habe vier Stationen...

[apollon77]

@johgre bitte nochmal von github updaten. Da gab es mal keinen response. Da hatte ich gestern Abend noch was gefixt.

[apollon77]

@ofbeqnpolkkl6mby5e13 und auch alles ok? Ist die Objekt Struktur auch sinnvoll so wie Sie jetzt ist?

[oFbEQnpoLKKl6mbY5e13]

@apollon77

Gute Frage, es wäre ein riesen Aufwand, die ganzen Aliase anzupassen, bei so vielen Modulen...
Müsste mir erst wieder ein Testsystem erstellen...
Im Moment leider wenig Zeit...

[oFbEQnpoLKKl6mbY5e13]

@apollon77

Puh... wenn man zwei Module mit der Bezeichnung Badezimmer hat, weiß man jetzt nicht mehr, welches welches ist (Obergeschoss, Erdgeschoss)... es muss unbedingt erst nach Station unterschieden werden!

Und das crashed nach dem Konfigurieren:

2022-09-08 22:44:35.676  - info: netatmo.0 (3932) starting. Version 2.0.0 (non-npm: PArns/ioBroker.netatmo#4fc7576c43e782e34bf464b63196a4af85a242d1) in /opt/iobroker/node_modules/iobroker.netatmo, node: v16.17.0, js-controller: 4.0.23
2022-09-08 22:44:35.722  - info: netatmo.0 (3932) Authenticating with Netatmo using individual client-ID xxx
2022-09-08 22:44:35.724  - warn: netatmo.0 (3932) API Error: Please Authenticate manually once using the Admin UI of this instance.
2022-09-08 22:47:26.586  - warn: netatmo.0 (3932) API Error: Authenticate "client_id" not set.
2022-09-08 22:47:31.624  - warn: netatmo.0 (3932) API Error: Authenticate "client_id" not set.
2022-09-08 22:47:36.631  - warn: netatmo.0 (3932) API Error: Authenticate "client_id" not set.
2022-09-08 22:47:41.691  - warn: netatmo.0 (3932) API Error: Authenticate "client_id" not set.
2022-09-08 22:47:53.588  - info: netatmo.0 (3932) Successfully authenticated with Netatmo with individual client-ID xxx
2022-09-08 22:47:53.597  - info: netatmo.0 (3932) Save OAuth data: {"access_token":"xxx","refresh_token":"xxx","scope":"read_station","client_id":"xxx"}
2022-09-08 22:47:53.623  - info: netatmo.0 (3932) Update data in adapter configuration ... restarting ...
2022-09-08 22:47:53.659  - info: host.iobroker-test stopInstance system.adapter.netatmo.0 (force=false, process=true)
2022-09-08 22:47:53.668  - info: netatmo.0 (3932) Got terminate signal TERMINATE_YOURSELF
2022-09-08 22:47:53.667  - info: host.iobroker-test stopInstance system.adapter.netatmo.0 send kill signal
2022-09-08 22:47:53.671  - info: netatmo.0 (3932) cleaned everything up...
2022-09-08 22:47:53.673  - info: netatmo.0 (3932) terminating
2022-09-08 22:47:53.677  - info: netatmo.0 (3932) Terminated (ADAPTER_REQUESTED_TERMINATION): Without reason
2022-09-08 22:47:54.196  - warn: netatmo.0 (3932) Could not perform strict object check of state netatmo.0.5e85b1dc9da0e8304e2becfc.70-ee-50-06-97-ea.Temperature.TemperatureMin: DB closed
2022-09-08 22:47:54.197  - warn: netatmo.0 (3932) get state error: Connection is closed.
2022-09-08 22:47:54.200  - error: netatmo.0 (3932) Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch().
2022-09-08 22:47:54.201  - error: netatmo.0 (3932) unhandled promise rejection: DB closed
2022-09-08 22:47:54.572  - error: netatmo.0 (3932) Error: DB closed
    at Redis.sendCommand (/opt/iobroker/node_modules/ioredis/built/redis/index.js:636:24)
    at Redis.get (/opt/iobroker/node_modules/ioredis/built/commander.js:122:25)
    at StateRedisClient.setState (/opt/iobroker/node_modules/@iobroker/db-states-redis/lib/states/statesInRedisClient.js:657:40)
    at Adapter.setState (/opt/iobroker/node_modules/@iobroker/js-controller-adapter/src/lib/adapter/adapter.js:6485:35)
2022-09-08 22:47:54.573  - error: netatmo.0 (3932) DB closed
2022-09-08 22:47:54.673  - info: host.iobroker-test stopInstance system.adapter.netatmo.0 killing pid 3932
2022-09-08 22:47:54.688  - info: host.iobroker-test instance system.adapter.netatmo.0 terminated with code 11 (ADAPTER_REQUESTED_TERMINATION)
2022-09-08 22:47:56.703  - info: host.iobroker-test instance system.adapter.netatmo.0 started with pid 4117
2022-09-08 22:47:58.928  - info: netatmo.0 (4117) starting. Version 2.0.0 (non-npm: PArns/ioBroker.netatmo#4fc7576c43e782e34bf464b63196a4af85a242d1) in /opt/iobroker/node_modules/iobroker.netatmo, node: v16.17.0, js-controller: 4.0.23
2022-09-08 22:47:58.985  - info: netatmo.0 (4117) Using stored tokens to initialize ... {"access_token":"xxx","refresh_token":"xxx","scope":"read_station","client_id":"xxx"}
2022-09-08 22:47:58.987  - info: netatmo.0 (4117) Authenticating with Netatmo using individual client-ID xxx
2022-09-08 22:47:59.245  - info: netatmo.0 (4117) Successfully authenticated with Netatmo with individual client-ID xxx

[apollon77]

@ofbeqnpolkkl6mby5e13 Ok, dann bitte vollständiges Debug Log mir per E-Mails senden und genau beschreiben was das issue ist.

An Sich sollte die struktur homeID.moduleId.weiteres sein, damit sollte an sich doch jede Station ne andere Module die haben oder? Früher war es doch dann aber schlimmer weil früher alle "Badezimmer" in eins geworfen wurden und damit hattest Du doch gar keine unterscheidung oder?

Zu dem "crash" ... Am ENde kein Crash sondern ein restart nach Konfigänderung. ja der DB fehler ist anzusehen

[oFbEQnpoLKKl6mbY5e13]

@apollon77

Bisher:

Spoiler

v2.0.0

Spoiler

[apollon77]

@ofbeqnpolkkl6mby5e13 ja ok ist mehr. Über die Objektnamens hast du das gleiche ja. Wo waren denn Module wie Nordseite und so bisher? Nur unter Haus. Richtig?

[oFbEQnpoLKKl6mbY5e13]

@apollon77

Nein, die sind unter den zugehörigen Stationen.

Unter Haus ist nichts Brauchbares:

Spoiler

Vielleicht sollte ich das näher erklären, da du selbst ja keine Station besitzt:

Pro Station können maximal 3 Innenmodule, 1 Außenmodul, 1 Windmesser und 1 Regenmesser angeschlossen werden. Wenn man also mehr als 3 zusätzliche Innenmodule benötigt (die Stationen selbst dienen auch als Innenmodul!), dann braucht man eine weitere Station, die nun wieder u. a. auch ein Außenmodul mitbringt. Daher Nord- und Südseite.
Bei der neuen Struktur kann man nun nicht mehr erkennen, zu welchen Stationen z. B. Badezimmer gehört.

[apollon77]

@ofbeqnpolkkl6mby5e13 ok. Damit volles debug log bitte. Muss schauen wie das auf der datenebene aussieht.

[Schimi]

Ich finde die Ordnerstruktur besser als vorher... Gerade wen man mal die Bezeichnung ändert sollte es nun ja trotzdem "gleich" bleiben

Da ich noch nicht "klingeln" darf (die kleine schläft ), kann ich nur dazusagen das der Name der Klingel nicht übernommen wird?

edit
Bekomme das im Log... vielleicht liegt da der Fehler beim "Namen"

netatmo.0
2022-09-11 15:35:05.075	warn	Object 5eea11c3c20743509886b280.70-ee-50-6e-bb-74 is invalid: obj.common.name has an invalid type! Expected "string" or "object", received "number"

[JohGre]

Bei mir läuft der Adapter nun seit 4 Tagen ohne Probleme. Finde auch das die Ordnerstruktur besser ist. War zwar einmal etwas Aufwand bzgl. der Scripte anpassen, aber halb so wild. Jetzt ist ja dann Ruhe damit.

[apollon77]

@schimi bitte debug log per email mit Verweis auf den post bitte

[apollon77]

@johgre ich bin noch dabei mir wegen Wetterstationen nochmal was anzusehen. Vllt ändert sich nochmal was. Poste aber hier.

[oFbEQnpoLKKl6mbY5e13]

@apollon77

Völlig egal, wieviel sich noch an der neuen Struktur ändert, ich habe über 300 Aliasse, die auf Datenpunkte des Netatmo-Adapters verweisen, soll ich die alle manuell anpassen?

[apollon77]

@ofbeqnpolkkl6mby5e13 megasorry aber ich fürchte das kommt auf die zu, ja in dem Fall doppelt sorry weil die alte Adapterversion Vorauss ab Oktober auch Nicht mehr tut, also bleiben ist dann auch keine Option.

[oFbEQnpoLKKl6mbY5e13]

@apollon77

Wie du weißt, hat das Eine (Objektstruktur) nichts mit dem Anderen (Authentifizierung) zu tun...

[apollon77]

@ofbeqnpolkkl6mby5e13 ja … wie aber oben beschrieben nutzte der Adapter bisher auch als deprecated gekennzeichnete api calls. Keine Ahnung wann die wegfallen. Und in diesen apis gibt es nur limitierte Geräte und einige wie der co2 Sensor oder doorbell sind nur in den neuen apis. Das zu mischen bringt noch mehr Chaos.

Weiterhin habe ich den Adapter jetzt quasi übernommen und die bisherige Struktur war sehr uneinheitlich, gewachsen, überschneidend was verschiedene Geräte anging und potentiell gefährlich für Konflikte wegen gleichen Namen. Jede Namensänderung in netatmo führt im Adapter zu einer neuen Struktur. All das hat mich dazu gebracht das Ganze neu zu strukturieren, Sindbads ich das auch verstehe und künftig weiter betreuen kann.

Das das jetzt mit der Authentifizierungsänderung zusammenfällt ist blöd aber lag daran das ich parallel am doorbell gearbeitet habe und auch letzte Features wie der co2 Sensor und rauchmelder nicht so taten wie sie sollte. Das hat sich alles zeitlich blöderweise überschnitten mit dem Termin when auth Änderung.

Hätte ich eine neue Version machen können basierend auf der letzten veröffentlichten (wo Dinge nicht richtig tun) mit nur neuer auth? Ja vllt … habe mich bisher aber dagegen entschieden weil es nichts wirklich daran ändert wie die Version danach aussehen wird und einige doorbell Änderungen Zum testen schon auf github waren. Wäre also ggf wieder ein schritt zurück für andere User……

Ich verstehe deinen Schmerz durchaus. Ich hoffe du kannst meine Sichtweise auch ein bissl nachvollziehen.

Ingo