home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
72.65k stars 30.41k forks source link

Overkiz Integration: fail to change modes for water heater #94628

Closed rembomy closed 1 year ago

rembomy commented 1 year ago

The problem

When I try to set any mode in the integration, from Eco to auto for example then it does give error. Changing Temperature does nothing and temperature stays same in water heater (even after hours of waiting). Atlantic water heater, Cozytouch app.

What version of Home Assistant Core has the issue?

core-2023.6.1

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant Container

Integration causing the issue

Overkiz

Link to integration documentation on our website

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

Diagnostics information

image

Example YAML snippet

No response

Anything in the logs that might be useful for us?

Logger: homeassistant.components.websocket_api.http.connection
Source: components/overkiz/water_heater_entities/domestic_hot_water_production.py:327
Integration: Home Assistant WebSocket API ([documentation](https://www.home-assistant.io/integrations/websocket_api), [issues](https://github.com/home-assistant/core/issues?q=is%3Aissue+is%3Aopen+label%3A%22integration%3A+websocket_api%22))
First occurred: 11:18:56 (4 occurrences)
Last logged: 11:32:19

[547657004944] 'auto'
[547878770192] 'auto'
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/components/websocket_api/commands.py", line 205, in handle_call_service
    await hass.services.async_call(
  File "/usr/src/homeassistant/homeassistant/core.py", line 1910, in async_call
    task.result()
  File "/usr/src/homeassistant/homeassistant/core.py", line 1950, in _execute_service
    await cast(Callable[[ServiceCall], Awaitable[None]], handler.job.target)(
  File "/usr/src/homeassistant/homeassistant/helpers/entity_component.py", line 226, in handle_service
    await service.entity_service_call(
  File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 811, in entity_service_call
    future.result()  # pop exception if have
    ^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 1034, in async_request_call
    await coro
  File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 851, in _handle_entity_call
    await result
  File "/usr/src/homeassistant/homeassistant/components/overkiz/water_heater_entities/domestic_hot_water_production.py", line 327, in async_set_operation_mode
    OverkizCommand.SET_DHW_MODE, self.operation_mode_to_overkiz[operation_mode]
                                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^
KeyError: 'auto'

Additional information

It also seems to be missing boost option and absence options that are available in the app, but this is not the main topic of this issue. It is able to read data, but not able to input anything back to the servers.

Also some other random error appeared

Logger: homeassistant.components.sensor
Source: components/sensor/__init__.py:496
Integration: Sensor ([documentation](https://www.home-assistant.io/integrations/sensor), [issues](https://github.com/home-assistant/core/issues?q=is%3Aissue+is%3Aopen+label%3A%22integration%3A+sensor%22))
First occurred: 13 June 2023 at 20:38:56 (2 occurrences)
Last logged: 11:32:39

Entity sensor.modbuslink_1_1_water_volume_estimation_at_40_degc_2 (<class 'homeassistant.components.overkiz.sensor.OverkizStateSensor'>) is using state class 'measurement' which is impossible considering device class ('volume') it is using; expected None or one of 'total', 'total_increasing'; Please update your configuration if your entity is manually configured, otherwise create a bug report at https://github.com/home-assistant/core/issues?q=is%3Aopen+is%3Aissue+label%3A%22integration%3A+overkiz%22
Entity sensor.modbuslink_1_1_water_volume_estimation_at_40_degc (<class 'homeassistant.components.overkiz.sensor.OverkizStateSensor'>) is using state class 'measurement' which is impossible considering device class ('volume') it is using; expected None or one of 'total', 'total_increasing'; Please update your configuration if your entity is manually configured, otherwise create a bug report at https://github.com/home-assistant/core/issues?q=is%3Aopen+is%3Aissue+label%3A%22integration%3A+overkiz%22
home-assistant[bot] commented 1 year ago

Hey there @imicknl, @vlebourl, @tetienne, @nyrodev, mind taking a look at this issue as it has been labeled with an integration (overkiz) you are listed as a code owner for? Thanks!

Code owner commands Code owners of `overkiz` can trigger bot actions by commenting: - `@home-assistant close` Closes the issue. - `@home-assistant rename Awesome new title` Renames the issue. - `@home-assistant reopen` Reopen the issue. - `@home-assistant unassign overkiz` Removes the current integration label and assignees on the issue, add the integration domain after the command.

(message by CodeOwnersMention)


overkiz documentation overkiz source (message by IssueLinks)

MassimoL06 commented 1 year ago

Hello, I believe you are facing same issue I already reported in February this year. https://github.com/home-assistant/core/issues/87417

rembomy commented 1 year ago

@MassimoL06 Indeed, sorry for not finding the issue before. It looks like the same issue, should I send my logs there and is there anything else I should contribute to make the investigation easier?

iMicknl commented 1 year ago

Duplicate of #87417. Please add any additional info there, if hasn't been already shared.