Bonjour,
suite a ma décision de supprimer proxmox avec ha en vm, et d’installer ha en direct sur le mini pc.
merci pour votre aide!!!
Bonjour,
oui, j’ai creer une sauvegarde avant la réinstallation.
Ok, faut que tu arrête proprement HA ( arrêt systeme ). Tu débranche ta machine du secteur quelque seconde et rebranche.
C’est un soucis qui existe depuis 2 ans presque et j’ai vu que ca été fixer sur le supervisor 2024.6.1 ( pas sortie , la 2024.6.2 est release mais pas déployer).
home-assistant:main ← home-assistant:addon-not-started-restore
opened 08:58PM - 28 May 24 UTC
<!--
You are amazing! Thanks for contributing to our project!
Please, DO N… OT DELETE ANY TEXT from this template! (unless instructed).
-->
## Proposed change
Full restore currently stops the docker events listener before it begins as part of `shutdown`. It is not restarted afterwards leaving the addons in completely unknown states beyond that point. This PR fixes that issue, the docker monitor is no longer stopped.
It also fixes a minor possible issue where addons were not loaded prior to being restarted during a restore due to an exception. The first is the real cause of these "unknown state after restore" issues though.
## Type of change
<!--
What type of change does your PR introduce to Home Assistant?
NOTE: Please, check only 1! box!
If your PR requires multiple boxes to be checked, you'll most likely need to
split it into multiple PRs. This makes things easier and faster to code review.
-->
- [ ] Dependency upgrade
- [x] Bugfix (non-breaking change which fixes an issue)
- [ ] New feature (which adds functionality to the supervisor)
- [ ] Breaking change (fix/feature causing existing functionality to break)
- [ ] Code quality improvements to existing code or addition of tests
## Additional information
<!--
Details are important, and help maintainers processing your PR.
Please be sure to fill out additional details, if applicable.
-->
- This PR fixes or closes issue: fixes #
- This PR is related to issue:
- Link to documentation pull request:
- Link to cli pull request:
## Checklist
<!--
Put an `x` in the boxes that apply. You can also fill these out after
creating the PR. If you're unsure about any of them, don't hesitate to ask.
We're here to help! This is simply a reminder of what we are going to look
for before merging your code.
-->
- [ ] 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]
- [ ] The code has been formatted using Ruff (`ruff format supervisor tests`)
- [ ] Tests have been added to verify that the new code works.
If API endpoints of add-on configuration are added/changed:
- [ ] Documentation added/updated for [developers.home-assistant.io][docs-repository]
<!--
Thank you for contributing <3
Below, some useful links you could explore:
-->
[dev-checklist]: https://developers.home-assistant.io/docs/en/development_checklist.html
[docs-repository]: https://github.com/home-assistant/developers.home-assistant
## Summary by CodeRabbit
- **Bug Fixes**
- Ensured add-ons are properly loaded before running by moving the check to a `finally` block.
- **Improvements**
- Improved system shutdown process by removing unnecessary stopping of docker monitoring.
- **Tests**
- Added tests to verify addon state monitoring after full and partial restore operations.
edit:@Krull56 , enfin c’est fixer
edit2:
1 « J'aime »
ok ca marche…je vais faire ca
bon j’ai fais le malin…je dois retrouver mon mp…sauf que je m’apercois que je ne l’ai pas noté…la loose.ca va m’apprendre tiens…
1 « J'aime »
