jeroenterheerdt / HADailySensor

Sensor for Home Assistant that gets reset at midnight
MIT License
70 stars 16 forks source link

custom integration 'daily' calls async_forward_entry_setup #62

Open armin-gh opened 2 months ago

armin-gh commented 2 months ago

Hi HA 2024.7.2 ( Core in Python Venv):

Reports on startup: Jul 25 16:55:59 smarthome hass[25572]: 2024-07-25 16:55:58.994 WARNING (MainThread) [homeassistant.helpers.frame] Detected that custom integration 'daily' calls async_forward_entry_setup for integration, daily with title: Fritz7490MaxDailyReceived and entry_id: 32a5fd4bfb392e14483c645628edf83f, which is deprecated and will stop working in Home Assistant 2025.6, await async_forward_entry_setups instead at custom_components/daily/__init__.py, line 113: hass.async_create_task(, please create a bug report at https://github.com/jeroenterheerdt/HADailySensor/issues

and

Jul 25 16:56:00 smarthome hass[25572]: 2024-07-25 16:56:00.059 WARNING (MainThread) [homeassistant.helpers.frame] Detected code that calls async_forward_entry_setup for integration daily with title: Fritz7490MaxDailyReceived and entry_id: 32a5fd4bfb392e14483c645628edf83f, 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.

quite some time left before this is no longer working, but I have not seen it reported yet here.

armin-gh commented 2 months ago

Hi Smart Irrigation? Not at all. I'm only running HADailySensor as custom integration, version is 2024.6.2

Armin

Which version of smart Irrigation are you running? We already fixed this AFAIK.

— Reply to this email directly, view it on GitHub https://github.com/jeroenterheerdt/HADailySensor/issues/62#issuecomment-2251742654, or unsubscribe https://github.com/notifications/unsubscribe-auth/ARCEIMSQLSKJGFX7GETJSKTZOGMMPAVCNFSM6AAAAABLOYWNHCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENJRG42DENRVGQ. You are receiving this because you authored the thread.Message ID: @.***>

jeroenterheerdt commented 2 months ago

Yeah sorry, my mistake. Other integration 😊

Get Outlook for Androidhttps://aka.ms/AAb9ysg


From: armin-gh @.> Sent: Thursday, July 25, 2024 10:03:50 PM To: jeroenterheerdt/HADailySensor @.> Cc: Subscribed @.***> Subject: Re: [jeroenterheerdt/HADailySensor] custom integration 'daily' calls async_forward_entry_setup (Issue #62)

Hi Smart Irrigation? Not at all. I'm only running HADailySensor as custom integration, version is 2024.6.2

Armin

Which version of smart Irrigation are you running? We already fixed this AFAIK.

— Reply to this email directly, view it on GitHub https://github.com/jeroenterheerdt/HADailySensor/issues/62#issuecomment-2251742654, or unsubscribe https://github.com/notifications/unsubscribe-auth/ARCEIMSQLSKJGFX7GETJSKTZOGMMPAVCNFSM6AAAAABLOYWNHCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENJRG42DENRVGQ. You are receiving this because you authored the thread.Message ID: @.***>

— Reply to this email directly, view it on GitHubhttps://github.com/jeroenterheerdt/HADailySensor/issues/62#issuecomment-2251979949 or unsubscribehttps://github.com/notifications/unsubscribe-auth/AB6PIPTH6MDZECQPYRM7LA3ZOHKDNBFKMF2HI4TJMJ2XIZLTSOBKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJDUOJ2WLJDOMFWWLLTXMF2GG2C7MFRXI2LWNF2HTAVFOZQWY5LFUVUXG43VMWSG4YLNMWVXI2DSMVQWIX3UPFYGLLDTOVRGUZLDORPXI6LQMWWES43TOVSUG33NNVSW45FGORXXA2LDOOJIFJDUPFYGLKTSMVYG643JORXXE6NFOZQWY5LFVEZDOMZTGMZTCOBYQKSHI6LQMWSWS43TOVS2K5TBNR2WLKRSGQZTAMRXGQ2DCMVHORZGSZ3HMVZKMY3SMVQXIZI. You are receiving this email because you are subscribed to this thread.

Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Bellavista commented 4 weeks ago

Hi @jeroenterheerdt i have a similar issue, how i can solve ? Tnk you

2024-08-28 12:35:58.848 WARNING (MainThread) [homeassistant.helpers.frame] Detected code that calls async_forward_entry_setup for integration daily with title: Temp Studio Simona Min and entry_id: df7d154c4ae1e38ce2ce592822f37260, 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.
Stack (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 674, in run_until_complete
    self.run_forever()
  File "/usr/local/lib/python3.12/asyncio/base_events.py", line 641, in run_forever
    self._run_once()
  File "/usr/local/lib/python3.12/asyncio/base_events.py", line 1990, 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/config_entries.py", line 2161, in async_forward_entry_setup
    _report_non_awaited_platform_forwards(entry, "async_forward_entry_setup")
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 1201, in _report_non_awaited_platform_forwards
    report(
  File "/usr/src/homeassistant/homeassistant/helpers/frame.py", line 151, in report
    _LOGGER.warning(msg, stack_info=True)
jeroenterheerdt commented 3 weeks ago

This is warning, not an error. We will fix it eventually.

altserg commented 2 weeks ago

I confirm the issue is present after HASS Core update to 2025.9.1

dan-s-github commented 3 days ago

created a pr to address this issue #64