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

fix timeout issue in HA #12

Closed
wants to merge 1 commit into from
Closed

fix timeout issue in HA #12

wants to merge 1 commit into from

Conversation

SenTzu01
Copy link

See issue #110140

@zxdavb
Copy link
Collaborator

zxdavb commented Feb 29, 2024

... maybe I could be talked into 30s.

In any case, the actual value should be up to the code using this library (i.e. HA).

@SenTzu01
Copy link
Author

From my experience I can tell you 30s will not suffice. 120s was not stable, so I decided to increase to 300 just to be sure.
I expect this issue to arise during bootup of HA on less performing HW, such as RPi3.
Is increasing the timeout from the caller even possible from your codebase?
Anyway, I created a workaround for my personal instance, just thought I'd share my 2 cents.........

@jbouwh
Copy link
Owner

jbouwh commented Jun 5, 2024

We have implemented a data coordinator in HA that can handle timeout errors much better. A longer timeout seems not to make sense for me.

@jbouwh jbouwh closed this Jun 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants