DurgNomis-drol / ha_toyota

Toyota Connected Services integration for Home Assistant.
MIT License
148 stars 32 forks source link

ha_toyota breaks logging to home-assistant.log #242

Closed pamae closed 7 months ago

pamae commented 9 months ago

As soon as ha_Toyota integration is loaded there are no more loggings in the file home-assistant.log from any integration. I can still see the errors through core log in cli but nothing in the home assistant log GUI from the moment toyota integration is loaded.

When i remove the toyota integration and restart home assistant the logs work again. Re-adding ha_toyota the logs in home-assistant.log stop working again

Using Enable debug logging downloads only de home-assistant.log file without any errors from the moment Toyota integration was loaded

as far i as i could trace back it started happening with version 2.0.0 and HA 2024.2.0

CM000n commented 9 months ago

Lol, thats funny, because we are changing nothing regarding Home Assistant logging in the custom component itself. Maybe it is because the logging config of the mytoyota lib which we introduced in the latest releases there, ist redirecting the logs somewhere.

Thanks for reporting

moonter45 commented 9 months ago

same here, nothing related to toyota in crushed home-assistant.log also nothing in "docker logs --follow homeassistant" output, I see only destroyed logging formatting.

CM000n commented 9 months ago

The mytoyota lib currently overrides the root logger, which is propably causing this issue. Will have a look into it tomorrow.

CM000n commented 8 months ago

Sry. I'll have a look at this soon, but unfortunately I'm still quite busy at the moment and haven't found the time yet.

pamae commented 7 months ago

Still not working for me after merge

things is tried

no logs in homeassistant.log from any integration as soon as ha-toyota is loaded. When ha_toyota is removed from the system the logs work perfect.

CM000n commented 7 months ago

That is clear. We still have to update the mytoyota package version here in the ha_toyota package. This does not happen automatically.

fliespl commented 7 months ago

@CM000n thanks for great work! Is this one scheduled to be deployed anytime soon? I would like to resolve some HA issues, which are currently hidden by this :)