dvd-dev/hilo

Nouveau warning dans le log 'WebsocketClient._clean_queue' was never awaited

Closed this issue · 5 comments

Version of the custom_component

2023.08.3 avec fork pyhilo

Describe the bug

Depuis quelque temps j'ai ce nouveau message là dans le log, je tente de comprendre pourquoi.

log

Logger: py.warnings
Source: src/python-hilo/pyhilo/websocket.py:301 
First occurred: 10:06:35 AM (1 occurrences) 
Last logged: 10:06:35 AM

/config/src/python-hilo/pyhilo/websocket.py:301: RuntimeWarning: coroutine 'WebsocketClient._clean_queue' was never awaited self._clean_queue()

Fix en essai.

L'ajout d'un await aux lignes 301 et 326 du websocket me donnent les nouvelles erreurs suivantes, je vois donc où ça se fait caller:

This error originated from a custom integration.

Logger: py.warnings
Source: custom_components/hilo/__init__.py:395 
Integration: Hilo (documentation, issues) 
First occurred: 8:12:37 AM (1 occurrences) 
Last logged: 8:12:37 AM

/usr/local/lib/python3.11/site-packages/pyhilo/websocket.py:326: RuntimeWarning: coroutine 'WebsocketClient._clean_queue' was never awaited self._clean_queue()
This error originated from a custom integration.

Logger: py.warnings
Source: custom_components/hilo/__init__.py:438 
Integration: Hilo (documentation, issues) 
First occurred: 8:12:38 AM (2 occurrences) 
Last logged: 8:12:43 AM

/usr/local/lib/python3.11/site-packages/pyhilo/websocket.py:301: RuntimeWarning: coroutine 'WebsocketClient._clean_queue' was never awaited self._clean_queue()

Je tente de comprendre pourquoi encore...

Pour ceux que ça intéresse, mes nouvelles erreurs viennent d'un code 18 de ma part.

manifest.json requiert python-hilo >= 2023.9.1,

"requirements": ["python-hilo>=2023.9.1"],

Mon fork est considéré comme 2023.8.3 même si je suis up to date @valleedelisle probablement du à la manière qu'on release sur pypi? Ça faisait que mon fork se faisait kicker out et la version 2023.9.1 se réinstallait. J'ai pu confirmer le tout avec pip show python-hilo après un restart de HA.

Donc, j'ai changé localement mon manifest.json pour >= 2023.8.3 pour pouvoir tester en local.

Une fois ce changement fait le warning dans le log n'apparaît plus au bootup de HA. Je check demain matin car @nlz242 semble dire que Hilo voit un paquet de connexions tous les matins à 7h15. Théoriquement je ne devrais pas le voir dans le log demain à cette heure-là.

Le disconnect de 7h15 n'a pas généré le warning.

nlz242 commented

Une fois ce changement fait le warning dans le log n'apparaît plus au bootup de HA. Je check demain matin car @nlz242 semble dire que Hilo voit un paquet de connexions tous les matins à 7h15.

On investigue à savoir pourquoi les connexions des tous les HA coupent (et donc se ré-établissent automatiquement) à 7:15 précis chaque matin. Évidemment, pas en lien directe avec le warning que tu avais (sinon que la déconnexion causait une instance de ton warning). Les deconnexions ne causent pas de problêmes côté HA, c'est juste un truc un peu suspect qu'on essaie de comprendre :)