Reduce Freebox router Raid warning to one occurence

[fthiery]

Proposed change

#95242 added RAID info to disks attached to Freebox routers, but fail the integration to start for people without disks or RAID (with a generic error (APIResponse: {"msg": "Erreur interne", "success": false, "error_code": "internal_error"}), its fix #97696 added try-catch with a warn, but in the update function (causing thousands of warnings to appear in the logs). It should be at init, then use a boolean or other to not warn more.

This PR sets a boolean in case of failure so that subsequent calls are not done anymore.

Models that seemingly do not support raid enumeration are

• Freebox Pop
• Freebox Revolution

However, i believe that this kind of model-specific handling should be done at the underlying level, and opened a corresponding discussion here: hacf-fr/freebox-api#567

Type of change

• Dependency upgrade
• Bugfix (non-breaking change which fixes an issue)
• New integration (thank you!)
• New feature (which adds functionality to an existing integration)
• Deprecation (breaking change to happen in the future)
• Breaking change (fix/feature causing existing functionality to break)
• Code quality improvements to existing code or addition of tests

Additional information

• This PR fixes or closes issue: fixes Freebox: Unable to enumerate raid disks WARN appears more than 10000 times since 2023.8.1 #98274
• This PR is related to issues:
  • Freebox: Unable to enumerate raid disks WARN appears more than 10000 times since 2023.8.1 #98274
  • Fix freebox enumerate raid disks #97696
  • Freebox Revolution has no RAID #97652

Checklist

• The code change is tested and works locally.
• Local tests pass. Your PR cannot be merged unless tests pass
• There is no commented out code in this PR.
• I have followed the [development checklist][dev-checklist]
• I have followed the [perfect PR recommendations][perfect-pr]
• The code has been formatted using Black (black --fast homeassistant tests)
• [] Tests have been added to verify that the new code works.

[home-assistant]

Hey there @hacf-fr, @Quentame, mind taking a look at this pull request as it has been labeled with an integration (freebox) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of freebox can trigger bot actions by commenting:

• @home-assistant close Closes the pull request.
• @home-assistant rename Awesome new title Renames the pull request.
• @home-assistant reopen Reopen the pull request.
• @home-assistant unassign freebox Removes the current integration label and assignees on the pull request, add the integration domain after the command.

[fthiery]

What bothers me is that the component supposed to abstract the router differences should be the library, not the HA component: https://github.com/hacf-fr/freebox-api

I opened a discussion there: hacf-fr/freebox-api#567

[home-assistant]

Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍

Learn more about our pull request process.

[fthiery]

I ended up following the style introduced by the method update_home_devices which is exactly the same usecase.

[MartinHjelmare]

Please don't amend or squash commits after review has started to make it easier for readers to track changes. Thanks!

[MartinHjelmare]

Thanks!

[Quentame]

A bit late to approve, but still: approved 🎉

Thanks @fthiery !