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

Plugwise Smile integration stopped working #79708

Closed
aquatix opened this issue Oct 6, 2022 · 17 comments · Fixed by #79831
Closed

Plugwise Smile integration stopped working #79708

aquatix opened this issue Oct 6, 2022 · 17 comments · Fixed by #79831
Assignees

Comments

@aquatix
Copy link

aquatix commented Oct 6, 2022

The problem

I discovered after updating to 2022.10.0 on hassos (RaspberryPi 3B) that our thermostat integration stopped working. Even after removing the integration, restarting HA, repeatedly trying to set up again, I keep getting an error:

Logger: homeassistant.config_entries
Source: components/plugwise/gateway.py:59
First occurred: 10:43:24 (1 occurrences)
Last logged: 10:43:24
Error setting up entry Smile for plugwise

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 365, in async_setup
    result = await component.async_setup_entry(hass, self)
  File "/usr/src/homeassistant/homeassistant/components/plugwise/__init__.py", line 13, in async_setup_entry
    return await async_setup_entry_gw(hass, entry)
  File "/usr/src/homeassistant/homeassistant/components/plugwise/gateway.py", line 59, in async_setup_entry_gw
    api.get_all_devices()
  File "/usr/local/lib/python3.10/site-packages/plugwise/smile.py", line 142, in get_all_devices
    self._all_device_data()
  File "/usr/local/lib/python3.10/site-packages/plugwise/smile.py", line 80, in _all_device_data
    data = self._get_device_data(device_id)
  File "/usr/local/lib/python3.10/site-packages/plugwise/smile.py", line 226, in _get_device_data
    device_data = self._get_appliance_data(dev_id)
  File "/usr/local/lib/python3.10/site-packages/plugwise/helper.py", line 908, in _get_appliance_data
    self._lortherm_cooling_active = data["modulation_level"] == 100
KeyError: 'modulation_level'

What version of Home Assistant Core has the issue?

2022.10.0

What was the last working version of Home Assistant Core?

2022.9.7

What type of installation are you running?

Home Assistant OS

Integration causing the issue

Plugwise

Link to integration documentation on our website

https://www.home-assistant.io/integrations/plugwise

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 365, in async_setup
    result = await component.async_setup_entry(hass, self)
  File "/usr/src/homeassistant/homeassistant/components/plugwise/__init__.py", line 13, in async_setup_entry
    return await async_setup_entry_gw(hass, entry)
  File "/usr/src/homeassistant/homeassistant/components/plugwise/gateway.py", line 59, in async_setup_entry_gw
    api.get_all_devices()
  File "/usr/local/lib/python3.10/site-packages/plugwise/smile.py", line 142, in get_all_devices
    self._all_device_data()
  File "/usr/local/lib/python3.10/site-packages/plugwise/smile.py", line 80, in _all_device_data
    data = self._get_device_data(device_id)
  File "/usr/local/lib/python3.10/site-packages/plugwise/smile.py", line 226, in _get_device_data
    device_data = self._get_appliance_data(dev_id)
  File "/usr/local/lib/python3.10/site-packages/plugwise/helper.py", line 908, in _get_appliance_data
    self._lortherm_cooling_active = data["modulation_level"] == 100
KeyError: 'modulation_level'


### Additional information

_No response_
@homeassistant
Copy link
Contributor

Hey there @CoMPaTech, @bouwew, @brefra, @frenck, mind taking a look at this issue as it has been labeled with an integration (plugwise) you are listed as a code owner for? Thanks!
(message by CodeOwnersMention)


plugwise documentation
plugwise source
(message by IssueLinks)

@aquatix
Copy link
Author

aquatix commented Oct 6, 2022

I verified that the Smile box is reachable on the IP address provided, it works fine; also, HA autodiscovers the integration, so that part works.

@bouwew
Copy link
Contributor

bouwew commented Oct 6, 2022

@aquatix What kind of device is connected to your Smile?
A CV-ketel of a heatpump? Of which brand/name?
Do you use a special Plugwise feature for that device?

@aquatix
Copy link
Author

aquatix commented Oct 6, 2022

A CV-ketel, no special features configured. Will look up the brand shortly.

It worked fine for years :)

@aquatix
Copy link
Author

aquatix commented Oct 6, 2022

The Smile is connected to an Anna thermostat by the way.

@aquatix
Copy link
Author

aquatix commented Oct 6, 2022

The CV is an Intergas for which the Smile measures boiler temp, set temp and such.

@bouwew
Copy link
Contributor

bouwew commented Oct 6, 2022

Ok, thanks. Very strange that this happens, but, maybe I made a mistake in the code somewhere.
I'll have a look at the code later today, after work and dinner :)

@bouwew
Copy link
Contributor

bouwew commented Oct 6, 2022

Can you download the Diagnostics of the Plugwise Smile (Anna) Integration and share the result with me?

@aquatix
Copy link
Author

aquatix commented Oct 7, 2022

I'd like to, but where do I find those? The add-on says it failed to set up, with only the provided logs as information.

@bouwew
Copy link
Contributor

bouwew commented Oct 7, 2022

Go to the Plugwise Integration tile under Settings --> Devices & Services --> Find the Smile Anna tile, press the 3 vertical points in the tile, press Download diagnostics.

@aquatix
Copy link
Author

aquatix commented Oct 7, 2022

Screenshot_20221007-082009.png

@bouwew
Copy link
Contributor

bouwew commented Oct 7, 2022

Ok that's not possible due to the integration not loading.

Next request: please capture the xml-data from http://<smile_ip>/core/domain_objects
Please make sure to save the data as xml, not as text. Can be done by right-click --> save as
Please send the file to my email: bouwe at westerdijk point info (make it a real email-address)

@aquatix
Copy link
Author

aquatix commented Oct 7, 2022

Sent! Thanks :)

@WhoSayIn
Copy link

WhoSayIn commented Oct 7, 2022

I have the same issue, my cv ketel is also an Intergas.

Can provide more information if needed.

@aquatix
Copy link
Author

aquatix commented Oct 7, 2022

@WhoSayIn I've had email contact with @bouwew to provide some XML dumps. It seems he now knows where the problem lays, and will try to fix it this evening.

bouwew added a commit to plugwise/python-plugwise that referenced this issue Oct 7, 2022
bouwew added a commit to plugwise/python-plugwise that referenced this issue Oct 7, 2022
bouwew added a commit to plugwise/python-plugwise that referenced this issue Oct 8, 2022
@AussiSG
Copy link

AussiSG commented Oct 17, 2022

Sorry to open up the thread again. But I still have the problem when I recently upgraded to 2022.10.4
Does this mean that it hasn't been implemented yet in the HA updates?

Getting the following (same) error :

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 365, in async_setup
    result = await component.async_setup_entry(hass, self)
  File "/usr/src/homeassistant/homeassistant/components/plugwise/__init__.py", line 13, in async_setup_entry
    return await async_setup_entry_gw(hass, entry)
  File "/usr/src/homeassistant/homeassistant/components/plugwise/gateway.py", line 59, in async_setup_entry_gw
    api.get_all_devices()
  File "/usr/local/lib/python3.10/site-packages/plugwise/smile.py", line 142, in get_all_devices
    self._all_device_data()
  File "/usr/local/lib/python3.10/site-packages/plugwise/smile.py", line 80, in _all_device_data
    data = self._get_device_data(device_id)
  File "/usr/local/lib/python3.10/site-packages/plugwise/smile.py", line 226, in _get_device_data
    device_data = self._get_appliance_data(dev_id)
  File "/usr/local/lib/python3.10/site-packages/plugwise/helper.py", line 908, in _get_appliance_data
    self._lortherm_cooling_active = data["modulation_level"] == 100
KeyError: 'modulation_level'

@frenck
Copy link
Member

frenck commented Oct 17, 2022

@AussiSG This issue has been closed and handled, could you pleas open an new issue instead? Thanks! 👍

@github-actions github-actions bot locked and limited conversation to collaborators Nov 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants