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
70.84k stars 29.54k forks source link

Detected blocking call to open inside the event loop by integration 'upb' #119083

Closed tdejneka closed 1 day ago

tdejneka commented 2 months ago

The problem

Received a warning message after startup that the UPB integration is performing a blocking call in the event loop.

Message suggests reporting the problem as an Issue.

What version of Home Assistant Core has the issue?

core-2024.6.0

What was the last working version of Home Assistant Core?

core-2024.5.4

What type of installation are you running?

Home Assistant Supervised

Integration causing the issue

UPB

Link to integration documentation on our website

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

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

Detected blocking call to open inside the event loop by integration 'upb' at homeassistant/components/upb/__init__.py,
 line 28: upb = upb_lib.UpbPim({"url": url, "UPStartExportFile": file}) 
 (offender: /usr/local/lib/python3.12/site-packages/upb_lib/parse_upstart.py, line 18: with open(filename) as file_handle:),
 please create a bug report at https://github.com/home-assistant/core/issues?q=is%3Aopen+is%3Aissue+label%3A%22integration%3A+upb%22

Traceback (most recent call last): File "<frozen runpy>",
 line 198, in _run_module_as_main File "<frozen runpy>",
 line 88, in _run_code File "/usr/src/homeassistant/homeassistant/__main__.py",
 line 223, in <module> sys.exit(main()) File "/usr/src/homeassistant/homeassistant/__main__.py",
 line 209, in main exit_code = runner.run(runtime_conf) File "/usr/src/homeassistant/homeassistant/runner.py",
 line 190, in run return loop.run_until_complete(setup_and_run_hass(runtime_config)) File "/usr/local/lib/python3.12/asyncio/base_events.py", 
 line 672, in run_until_complete self.run_forever() File "/usr/local/lib/python3.12/asyncio/base_events.py", 
 line 639, in run_forever self._run_once() File "/usr/local/lib/python3.12/asyncio/base_events.py", 
 line 1988, in _run_once handle._run() File "/usr/local/lib/python3.12/asyncio/events.py", 
 line 88, in _run self._context.run(self._callback, *self._args) File "/usr/src/homeassistant/homeassistant/setup.py", 
 line 165, in async_setup_component result = await _async_setup_component(hass, domain, config) File "/usr/src/homeassistant/homeassistant/setup.py", 
 line 447, in _async_setup_component await asyncio.gather( File "/usr/src/homeassistant/homeassistant/setup.py", 
 line 449, in <genexpr> create_eager_task( File "/usr/src/homeassistant/homeassistant/util/async_.py", 
 line 37, in create_eager_task return Task(coro, loop=loop, name=name, eager_start=True) File "/usr/src/homeassistant/homeassistant/config_entries.py", 
 line 742, in async_setup_locked await self.async_setup(hass, integration=integration) File "/usr/src/homeassistant/homeassistant/config_entries.py", 
 line 594, in async_setup result = await component.async_setup_entry(hass, self) File "/usr/src/homeassistant/homeassistant/components/upb/__init__.py", 
 line 28, in async_setup_entry upb = upb_lib.UpbPim({"url": url, "UPStartExportFile": file})

Additional information

No response

home-assistant[bot] commented 2 months ago

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

Code owner commands Code owners of `upb` 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 upb` Removes the current integration label and assignees on the issue, add the integration domain after the command. - `@home-assistant add-label needs-more-information` Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue. - `@home-assistant remove-label needs-more-information` Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


upb documentation upb source (message by IssueLinks)

gwww commented 2 months ago

I should have a fix next week.

bdraco commented 1 day ago

fixed via https://github.com/home-assistant/core/pull/120689