bigmoby / fglair_for_homeassistant

❄️ FGLair heat pump controller integration
MIT License
48 stars 4 forks source link

WARNINGS on v2024.6.0 #133

Closed Korvenwin closed 3 months ago

Korvenwin commented 3 months ago

System Health details

System Information

version | core-2024.6.0

Checklist

Describe the issue

After update to v2024.6.0, the log show some warnings.

Reproduction steps

  1. Check log viewer

Debug logs

`2024-06-06 09:40:51.626 WARNING (MainThread) [homeassistant.util.loop] Detected blocking call to open inside the event loop by custom integration 'fglair_heatpump_controller' at custom_components/fglair_heatpump_controller/climate.py, line 414: self._properties = await self._fujitsu_device.async_update_properties() (offender: /usr/local/lib/python3.12/site-packages/pyfujitsugeneral/client.py, line 87: f = open(access_token_file, encoding="utf-8")), please create a bug report at https://github.com/bigmoby/fglair_for_homeassistant/issues`

`2024-06-06 09:40:52.070 WARNING (MainThread) [homeassistant.util.loop] Detected blocking call to open inside the event loop by custom integration 'fglair_heatpump_controller' at custom_components/fglair_heatpump_controller/climate.py, line 414: self._properties = await self._fujitsu_device.async_update_properties() (offender: /usr/local/lib/python3.12/site-packages/pyfujitsugeneral/client.py, line 181: with open(self._ACCESS_TOKEN_FILE, "w", encoding="utf-8") as f:), please create a bug report at https://github.com/bigmoby/fglair_for_homeassistant/issues`

Diagnostics dump

No response

colino17 commented 3 months ago

I am also getting this error in my HA logs. I also noticed my server had a few system reboots that I couldn't explain. The journalctl logging of reboot reasons showed me the HA logs related to this component.

Korvenwin commented 3 months ago

In my case, everything works fine without any issue. Only warnings on log.

I should say that this is not the only integration with warnings.

bigmoby commented 3 months ago

The release https://github.com/bigmoby/fglair_for_homeassistant/releases/tag/v0.3.13 should fix. Thanks to @drc38 for fix 🙌

Korvenwin commented 3 months ago

THANK YOU VERY MUCH. Your great support is the cause why I've bought another Fujitsu Air conditioning equipment with FGLair support (although it appears that new models came with Airstage support).

Korvenwin commented 2 months ago

Hi, again. Warnings are missing but now I receive an excpetion, although the integration appears to works fine:

File "/usr/local/lib/python3.12/site-packages/pyfujitsugeneral/client.py", line 238, in api_wrapper raise FGLairGeneralException from exception pyfujitsugeneral.exceptions.FGLairGeneralException: FGLairBaseException has been raised

EDIT: After start de A/A and stop it, the error is missing.

bigmoby commented 2 months ago

Hi @Korvenwin how could I reproduce this exception? Please could you write a clear step by step procedure?

Korvenwin commented 2 months ago

I'm sorry, after operate with the equipment the exceptdion dissapeared and I haven't gotten it any more.

bigmoby commented 2 months ago

Ok no problem, thanks 👍