joggs / home_assistant_ebeco

Integration for Ebeco thermostats
29 stars 14 forks source link

Adding new thermostats gives Unkown error #27

Open MacMac1 opened 1 year ago

MacMac1 commented 1 year ago

You can add only one EB-500 thermostat at a time. After adding a thermostat the UI will give you Unknown error and you need to restart the whole HA before adding the next one. image

GlimSo commented 1 year ago

Have the exact same issue.

oppigard commented 1 year ago

Same

Turbobussi commented 1 year ago

Same here.

joggs commented 1 year ago

@PerfectlyNormal . Can you take a look?

PerfectlyNormal commented 1 year ago

After restarting, it works to add a new one and the integration works as expected? It's just the initial setup that's wonky?

Is there anything relevant being shown in the logs? Under Settings in the left menu, then System, then Logs.

I only have a single thermostat to test with, so support for multiple is very theoretical from my side. Which means I can't really reproduce the problem and will have to rely on a bit of guesswork

Turbobussi commented 1 year ago

After adding one reboot and second one working. But adding third have to reboot again.

Here is code: `Logger: aiohttp.server Source: data_entry_flow.py:223 First occurred: 21:17:12 (3 occurrences) Last logged: 21:19:20

Error handling request Traceback (most recent call last): File "/usr/local/lib/python3.10/site-packages/aiohttp/web_protocol.py", line 433, in _handle_request resp = await request_handler(request) File "/usr/local/lib/python3.10/site-packages/aiohttp/web_app.py", line 504, in _handle resp = await handler(request) File "/usr/local/lib/python3.10/site-packages/aiohttp/web_middlewares.py", line 117, in impl return await handler(request) File "/usr/src/homeassistant/homeassistant/components/http/security_filter.py", line 85, in security_filter_middleware return await handler(request) File "/usr/src/homeassistant/homeassistant/components/http/forwarded.py", line 100, in forwarded_middleware return await handler(request) File "/usr/src/homeassistant/homeassistant/components/http/request_context.py", line 28, in request_context_middleware return await handler(request) File "/usr/src/homeassistant/homeassistant/components/http/ban.py", line 80, in ban_middleware return await handler(request) File "/usr/src/homeassistant/homeassistant/components/http/auth.py", line 235, in auth_middleware return await handler(request) File "/usr/src/homeassistant/homeassistant/components/http/view.py", line 146, in handle result = await result File "/usr/src/homeassistant/homeassistant/components/config/config_entries.py", line 86, in delete result = await hass.config_entries.async_remove(entry_id) File "/usr/src/homeassistant/homeassistant/config_entries.py", line 1088, in async_remove for progress_flow in self.hass.config_entries.flow.async_progress_by_handler( File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 199, in async_progress_by_handler self._async_progress_by_handler(handler), include_uninitialized File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 222, in _async_progress_by_handler return [ File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 223, in self._progress[flow_id] KeyError: 'e4b39dc12bb245bffc73a4064b25c99a' `

velodromen commented 1 year ago

Same issue here. Just installed it and managed to add one unit and then I received the Unknown error when trying to add a second unit. I have five units and I got the error message irregardless of which of the units i picked. Will try the workaround suggested (restart HA to add additional units.)

PerfectlyNormal commented 9 months ago

There's potentially a fix in #33 (in 336b264). Stepping through the code took me into some internal HomeAssistant cleanup that threw this error. Renaming a variable (🤦) makes it work better for me, but since I don't have multiple devices available, I can't test it properly yet

joggs commented 9 months ago

There's potentially a fix in #33 (in 336b264). Stepping through the code took me into some internal HomeAssistant cleanup that threw this error. Renaming a variable (🤦) makes it work better for me, but since I don't have multiple devices available, I can't test it properly yet

Great. We'll give it a shot. A new release has been created with these changes