-
-
Notifications
You must be signed in to change notification settings - Fork 32.8k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Reduce polling rate in Blink #107386
Reduce polling rate in Blink #107386
Conversation
Hey there @fronzbot, mind taking a look at this pull request as it has been labeled with an integration ( Code owner commandsCode owners of
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Problem seems to be solved by changing to user agent fronzbot/blinkpy#848 |
Let's hold off for a bit. |
Yeah I agree with @mkmer. User agent fixes the problem, but it's best we increase the polling time as well |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @mkmer 👍
../Frenck
Proposed change
Revert polling rate to previous default 300s
While this was running for a least 2 months, since 2024.1.0 many users have been receiving unexpected 406 errors from the API. Since it may be a resource constraint on Blink side that appeared after HA reached "critical mass" on updates, I think we should revert to pre 2023.11 rate and re-evaluate.
Please tag for next bug fix release.
Type of change
Additional information
Checklist
ruff format homeassistant tests
)If user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
Updated and included derived files by running:
python3 -m script.hassfest
.requirements_all.txt
.Updated by running
python3 -m script.gen_requirements_all
..coveragerc
.To help with the load of incoming pull requests: