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

HA Warning: async_forward_entry_setup lock issue #157

Open
ProphetOfDoom opened this issue Jul 18, 2024 · 3 comments
Open

HA Warning: async_forward_entry_setup lock issue #157

ProphetOfDoom opened this issue Jul 18, 2024 · 3 comments
Assignees
Labels
bug Something isn't working

Comments

@ProphetOfDoom
Copy link

Describe the bug
async_forward_entry_setup can cause the setup lock to be released before the setup is done and will stop working in HA 2025.1

To Reproduce
Start/Restart Home Assistant
Check system log

Expected behaviour
Warning should not appear

Screenshots
HA Version Info:

Screenshot 2024-07-17 163141

Desktop (please complete the following information):
N/A

Additional context
System log entry:

Logger: homeassistant.helpers.frame
Source: helpers/frame.py:151
First occurred: 16:06:49 (5 occurrences)
Last logged: 16:06:49

Detected code that calls async_forward_entry_setup for integration hive with title: and entry_id: , during setup without awaiting async_forward_entry_setup, which can cause the setup lock to be released before the setup is done. This will stop working in Home Assistant 2025.1. Please report this issue.

@ProphetOfDoom ProphetOfDoom added the bug Something isn't working label Jul 18, 2024
@Nuuki9
Copy link

Nuuki9 commented Jan 2, 2025

Has there been any news or feedback on this issue? 2025.1 releases tomorrow I believe, so knowing if that will break this integration would be good to know in advance.

@ProphetOfDoom
Copy link
Author

Has there been any news or feedback on this issue? 2025.1 releases tomorrow I believe, so knowing if that will break this integration would be good to know in advance.

As the person who originally raised this issue, it's only fair that I point out that I have since swapped back to using the HA Hive Integration as it had a major upgrade a few months ago and now gives me all the info I felt was lacking.

@Nuuki9
Copy link

Nuuki9 commented Jan 2, 2025

Good to know. I'm incredibly grateful for the work of the project owner (and all custom integrations), but if the core integration has addressed some of the original gaps then using that would be ideal.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants