-
-
Notifications
You must be signed in to change notification settings - Fork 30.3k
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
Freebox: Unable to enumerate raid disks WARN appears more than 10000 times since 2023.8.1 #98274
Comments
Hey there @hacf-fr, mind taking a look at this issue as it has been labeled with an integration ( Code owner commandsCode owners of
(message by CodeOwnersMention) freebox documentation |
This comment was marked as spam.
This comment was marked as spam.
This comment was marked as spam.
This comment was marked as spam.
This comment was marked as spam.
This comment was marked as spam.
Hi, really sorry for this regression, it was not possible for me to simulate a different freebox model, and nothing indicated that the freebox API is not the same :( I have opened a PR here: #98740 I'm not sure whether we should try to mock the error for simulated different router models... Since i only own a Delta model, can anyone with e.g. a Freebox Revolution try my PR ? As a reminder, to test it, you need to upload the homeasistant/components/freebox folder into your |
…n raised an http error, fixes home-assistant#98274
* consider Freebox router does not support Raid if the first enumeration raised an http error, fixes #98274 * add router name to warning message * reduce log level to info, remove details
* consider Freebox router does not support Raid if the first enumeration raised an http error, fixes #98274 * add router name to warning message * reduce log level to info, remove details
Hi @fthiery ! No worries, I know those kind of things can happen with different hardware having specificities and acting differently. I have a Freebox Revolution, will test on dev if I find time, otherwise 2023.8.4 Thanks ! EDIT: working well, displaying the error message only once. |
The problem
#95242 added RAID info to disks attached to Freebox routers, but fail the integration to start for people without disks or RAID, its fix #97696 added try-catch with a warn, but in the update function. It should be at init, then use a boolean or other to not warn more.
Actual behavior: WARN log infinitaly, at every update
Expected behavior: WARN log only once, at initialisation
What version of Home Assistant Core has the issue?
core-2023.8.1
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
freebox
Link to integration documentation on our website
https://www.home-assistant.io/integrations/freebox/
Diagnostics information
No response
Example YAML snippet
No response
Anything in the logs that might be useful for us?
Additional information
No response
The text was updated successfully, but these errors were encountered: