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 logs growing quickly with Owl intuition warnings #32

Open
CarlNightingale opened this issue Jul 1, 2023 · 7 comments
Open

HA logs growing quickly with Owl intuition warnings #32

CarlNightingale opened this issue Jul 1, 2023 · 7 comments

Comments

@CarlNightingale
Copy link

Hi, Please see the attached log screenshots. This seemed to start after I updated to the latest patch a few days ago. Is the anyway to prevent the warnings being written to the logs or to change the parameters to reduce the number of records please?

Thanks in advance

log Screenshot 2023-07-01 170101
log1 Screenshot 2023-07-01 170001

@glpatcern
Copy link
Member

Hi there,
Missed this earlier, sorry. For the Impossible energy consumption measure errors, I'll have a look as it seems coming from the sensor - and it may well be that things changed in HA.

For the other warnings, they come from HA core and are due to the fact that this integration gets updates synchronously (and "slowly", OWL sends an update every 30-60 seconds). Some years ago I attempted to go async but that made HA totally unstable, I was likely not doing the right thing and/or the async support in HA was not yet mature enough. I never tried again afterwards and can't promise anything at this stage. Of course I'd be happy to review a contribution in a PR for this feature.

@shortbloke
Copy link
Contributor

You may also want to look at your logger configuration, perhaps changing your default level to error and then specifying higher log levels for components and services you're troubleshooting.

@glpatcern
Copy link
Member

@tluethi71
Copy link

For (my) future reference, home-assistant/[email protected]:core:govee-local-api#diff-928dc93a5708a4616e87c0cd2f2ac47141bdb03d1226e8bc5c2eea2b41f3d8f1R27-R41 is a new integration in HA that uses asyncio with the HA main loop, and https://github.com/Galorhallen/govee-local-api/blob/develop/src/govee_local_api/controller.py#L97-L100 is how the listener is initialized

Will this solve the long startup procedure? I had a look at it but honestly I dont understand what I have to do...

@glpatcern
Copy link
Member

Will this solve the long startup procedure? I had a look at it but honestly I dont understand what I have to do...

This might solve it, yes, but it requires a major refactoring of the code and a developer-level understanding of async loops.

I may have time to review a PR on this, but can't commit myself to such task for now.

@tluethi71
Copy link

hey @glpatcern grazie mille. ;) then I will wait. I had to uninstalled it as every reboot is taking so long and I dont know how I can get rid of the owl-startup message. I only use it for my triphase owl sensor...

@glpatcern
Copy link
Member

Prego ;) the comment above suggests to change the log level, for the startup time I also use it for the triphase and I find it acceptable (the rest of HA normally runs), then of course it's your choice

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

No branches or pull requests

4 participants