dwradcliffe / intellicenter

Home Assistant Integration for Pentair Intellicenter
GNU General Public License v3.0
6 stars 1 forks source link

async_forward_entry_setup deprecated #8

Closed yestrdave closed 2 weeks ago

yestrdave commented 2 months ago

Logger: homeassistant.helpers.frame Source: helpers/frame.py:151 First occurred: 9:09:34 PM (11 occurrences) Last logged: 9:09:44 PM

Detected code that calls async_forward_entry_setup for integration, intellicenter with title: Pentair Pool and entry_id: xxxx, which is deprecated and will stop working in Home Assistant 2025.6, await async_forward_entry_setups instead. Please report this issue. Detected code that calls async_forward_entry_setup for integration intellicenter with title: Pentair Pool and entry_id: xxxx, 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.

henryr01 commented 1 month ago

This just turned into a Present Issue! The integration stopped working today with the 2024.8.3 update. :(

dwradcliffe commented 1 month ago

Shoot. I’ll try to take a look tonight.


David Radcliffe

On Mon, Aug 26, 2024 at 5:06 PM henryr01 @.***> wrote:

This just turned into a Present Issue! The integration stopped working today with the 2024.8.3 update. :(

— Reply to this email directly, view it on GitHub https://github.com/dwradcliffe/intellicenter/issues/8#issuecomment-2311096933 or unsubscribe https://github.com/notifications/unsubscribe-auth/AAA5AQQOD46LJRNR4KCZP43ZTOKFRBFKMF2HI4TJMJ2XIZLTSOBKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJDUOJ2WLJDOMFWWLLTXMF2GG2C7MFRXI2LWNF2HTAVFOZQWY5LFUVUXG43VMWSG4YLNMWVXI2DSMVQWIX3UPFYGLLDTOVRGUZLDORPXI6LQMWWES43TOVSUG33NNVSW45FGORXXA2LDOOJIFJDUPFYGLKTSMVYG643JORXXE6NFOZQWY5LFVEZTKNBRGI4TINBZQKSHI6LQMWSWS43TOVS2K5TBNR2WLKRSGQYDSOJWGMYTCMFHORZGSZ3HMVZKMY3SMVQXIZI . You are receiving this email because you are subscribed to this thread.

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

henryr01 commented 1 month ago

Multiple restarts and reloads, still wouldn't connect. Deleted the integration, then re-added it. It took a few minutes, but found everything now. Not sure what happened. I captured the logs below when it wouldn't load.

2024-08-26 17:23:46.890 ERROR (MainThread) [custom_components.intellicenter.pyintellicenter.controller] cannot start: [Errno 113] Connect call failed ('192.168.0.69', 6681) 2024-08-26 17:23:46.890 INFO (MainThread) [custom_components.intellicenter.pyintellicenter.controller] will attempt to reconnect in 30s 2024-08-26 17:24:13.284 ERROR (SyncWorker_4) [total_connect_client.client] exception during initial fetch of 2654404 2024-08-26 17:24:17.922 INFO (MainThread) [custom_components.intellicenter.pyintellicenter.controller] model now contains 76 objects 2024-08-26 17:24:18.409 WARNING (MainThread) [custom_components.intellicenter.pyintellicenter.controller] CONTROLLER: error 400 : {'command': 'Error', 'messageID': 'e610fb19-0ad6-424c-a1cd-f9964df190bb', 'response': '400', 'description': 'ParseError: unexpected end of input in string'} 2024-08-26 17:24:18.509 WARNING (MainThread) [custom_components.intellicenter.pyintellicenter.controller] CONTROLLER: error 400 : {'command': 'Error', 'messageID': 'd3d68660-326e-4911-a8b7-d55ae429edc0', 'response': '400', 'description': "ParseError: expected value, got 'U' (85)"}

dwradcliffe commented 1 month ago

I don't think your problem is related to the original warning. It looks like your intellicenter refused the connection and then returned a 400 error. Maybe it was in service mode or just had network issues for a minute?

dwradcliffe commented 2 weeks ago

Fixed in https://github.com/dwradcliffe/intellicenter/commit/cfc9cc82fe49697bed9386d229c547abca60d3ba