Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ha_toyota breaks logging to home-assistant.log #242

Closed
pamae opened this issue Feb 18, 2024 · 7 comments · Fixed by #251
Closed

ha_toyota breaks logging to home-assistant.log #242

pamae opened this issue Feb 18, 2024 · 7 comments · Fixed by #251
Labels

Comments

@pamae
Copy link

pamae commented Feb 18, 2024

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

@pamae pamae added the 🐛 bug label Feb 18, 2024
@CM000n
Copy link
Collaborator

CM000n commented Feb 18, 2024

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
Copy link

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
Copy link
Collaborator

CM000n commented Feb 18, 2024

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

@CM000n
Copy link
Collaborator

CM000n commented Mar 7, 2024

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
Copy link
Author

pamae commented Mar 27, 2024

Still not working for me after merge

things is tried

  • Restarted multiple times
  • full shutdown and start homeassistant
  • remove toyota integration, restart homeassistant add toyota integration
  • download lastest master from ha_toyota

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
Copy link
Collaborator

CM000n commented Mar 27, 2024

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

@fliespl
Copy link

fliespl commented Apr 4, 2024

@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 :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants