thomasloven / hass-plejd

🔹 Plejd BLE integration for Home Assistant
81 stars 10 forks source link

Integration stopped working after upgrade #43

Closed johanschelin closed 8 months ago

johanschelin commented 8 months ago

After upgrade to 3.0.3 the integration wont start. Even downgrading via HACS to 3.0.1. or 3.0.0 fails with same error.

Below from HA log, and also attaching a debug log. Is it something in my end, or did something break in the update?

Logger: homeassistant.config_entries Source: config_entries.py:399 First occurred: 00:00:21 (4 occurrences) Last logged: 00:24:26

Error setting up entry Karpvägen 26 (19 devices) for plejd Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/config_entries.py", line 399, in async_setup result = await component.async_setup_entry(hass, self) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/config/custom_components/plejd/init.py", line 29, in async_setup_entry await plejdManager.init() File "/usr/local/lib/python3.11/site-packages/pyplejd/init.py", line 48, in init self.devices = self.cloud.devices ^^^^^^^^^^^^^^^^^^ File "/usr/local/lib/python3.11/site-packages/pyplejd/cloud/init.py", line 159, in devices outputType = const.DEVICES.HARDWARE_TYPE[hardware]


KeyError: '-unknown- (19)'

[home-assistant_plejd_2023-10-03T22-24-29.186Z.log](https://github.com/thomasloven/hass_plejd/files/12797370/home-assistant_plejd_2023-10-03T22-24-29.186Z.log)
johanschelin commented 8 months ago

Partial restore of the HA Core got it up and running again - going back to 3.0.1 together with the rest of the Core.

thomasloven commented 8 months ago

This should be fixed now.

thomasloven commented 8 months ago

Actually, it probably isn't fixed at all... I'll make a new release momentarily

johanschelin commented 8 months ago

Many thanks Thomas, can confirm that my issue is solved! Great work

From: Thomas Lovén @.> Sent: 11 October 2023 21:56 To: thomasloven/hass_plejd @.> Cc: Johan Schelin @.>; Author @.> Subject: Re: [thomasloven/hass_plejd] Integration stopped working after upgrade (Issue #43)

Closed #43https://github.com/thomasloven/hass_plejd/issues/43 as completed via 4d21692https://github.com/thomasloven/hass_plejd/commit/4d21692503bea164cb09cdd884719fc010c51e0f.

— Reply to this email directly, view it on GitHubhttps://github.com/thomasloven/hass_plejd/issues/43#event-10623471586, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APTUQJUAMTGHJQDAWQURJCTX632TFANCNFSM6AAAAAA5RVNAH4. You are receiving this because you authored the thread.Message ID: @.**@.>>

thomasloven commented 8 months ago

You may or may not have a new device of model -unknown- (19). It could be interesting for the future to know if you happen to know what that is?

thomasloven commented 8 months ago

Thanks! I deleted your comment since the debugging data contains your address.

johanschelin commented 8 months ago

Ouch, should have read more carefully. Thanks for deletion!

From: Thomas Lovén @.> Sent: 11 October 2023 22:29 To: thomasloven/hass_plejd @.> Cc: Johan Schelin @.>; Author @.> Subject: Re: [thomasloven/hass_plejd] Integration stopped working after upgrade (Issue #43)

Thanks! I deleted your comment since the debugging data contains your address.

— Reply to this email directly, view it on GitHubhttps://github.com/thomasloven/hass_plejd/issues/43#issuecomment-1758485310, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APTUQJQLOJHNGQBO6KRPKMDX636P7ANCNFSM6AAAAAA5RVNAH4. You are receiving this because you authored the thread.Message ID: @.**@.>>