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
73.49k stars 30.7k forks source link

Pentair ScreenLogic integration requires frequent reloading to continue working #91626

Closed dadoftwins closed 1 year ago

dadoftwins commented 1 year ago

The problem

Since I upgraded from the last 2022 HomeAssistant release the Pentair ScreenLogic integration stops updating. Once it gets into a bad state the only thing that fixes it is reloading the integration. Once it's reloaded it works as expected for a day or two and then goes into a bad state again.

This is new behavior since updating -- it used to be rock solid!

When the integration is in a bad state, the Pentair ScreenLogic app still functions normally.

Please just let me know if there's any additional information I can provide!

What version of Home Assistant Core has the issue?

core-2023.3.6

What was the last working version of Home Assistant Core?

core-2022.12.?

What type of installation are you running?

Home Assistant OS

Integration causing the issue

Pentair Screenlogic

Link to integration documentation on our website

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

Diagnostics information

config_entry-screenlogic-b333ba56078657b34dbb7a713400234d.json.txt

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

home-assistant[bot] commented 1 year ago

Hey there @dieselrabbit, @bdraco, mind taking a look at this issue as it has been labeled with an integration (screenlogic) you are listed as a code owner for? Thanks!

Code owner commands Code owners of `screenlogic` 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 screenlogic` Removes the current integration label and assignees on the issue, add the integration domain after the command.

(message by CodeOwnersMention)


screenlogic documentation screenlogic source (message by IssueLinks)

dieselrabbit commented 1 year ago

The current requirement for screenlogicpy==0.8.2

Your diagnostic data lists the requirement as 0.7.2, which would exhibit this behavior in certain configurations.

I'm not sure why your integration is out of date, when you seem to have updated to the latest HA release.

dadoftwins commented 1 year ago

@dieselrabbit I think this might be fixed now! I updated to 2023.4.5 and when I check the diagnostic data the requirements correctly show 0.8.2 now. It's been a couple days and it's still working great.

dieselrabbit commented 1 year ago

@home-assistant close