-
-
Notifications
You must be signed in to change notification settings - Fork 28.5k
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
Settings -> Repairs -> Integration startup time is shown as negative #117530
Comments
see #116221 |
If HA allows for platforms "get loaded multiple times" - then may be HA should consider these possible cases? |
It's not possible to detect every incorrect way that an integration may write their code. |
Agree. See, I know nothing about HA internals, my proposals are only based on you explanation. |
It wouldn’t make sense to try to work out the timings for a component that doesn’t await its platform setups since it has a much larger problem that it could get reloaded while its platforms are being setup. Once the component fixes that the timings will be correct.Since the custom component runs at the same privilege level as HA, there is nothing we can do to prevent a custom component from loading platforms without awaiting their setup. IMHO, it’s a good thing that it returns a nonsensical result since it makes it obvious there is a problem with the custom component that needs to be fixed as otherwise it would have probably gone unnoticed Sent from my iPhoneOn May 17, 2024, at 2:23 AM, ildar170975 ***@***.***> wrote:
t's not possible to detect every incorrect way that an integration may write their code.
Agree.
But my point is:
-- since some platform may be "get loaded multiple times" - then this could happen since HA allows to do it "multiple times";
-- then either HA should disable this possibility (i.e. a platform may be loaded one time only);
-- or HA should consider these "get loaded multiple times" cases as a possible scenario when calculating a "startup time".
See, I know nothing about HA internals, my proposals are only based on you explanation.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: ***@***.***>
|
Thanks for explanation. |
@bdraco For the record - issue for Gismeteo integration - Limych/ha-gismeteo#191 |
Yes |
Again for the record - the author of Gismeteo integration declines that this "negative startup time" could be a "integration issue".
|
The problem
Settings -> Repairs -> Integration startup time
a negative value is shown:
What version of Home Assistant Core has the issue?
2024.5.3
What was the last working version of Home Assistant Core?
No response
What type of installation are you running?
Home Assistant Container
Integration causing the issue
No response
Link to integration documentation on our website
No response
Diagnostics information
No response
Example YAML snippet
No response
Anything in the logs that might be useful for us?
No response
Additional information
No response
The text was updated successfully, but these errors were encountered: