Skip to content

Commit 2602405

Browse files
authored
Fix add-on store repository getting removed without internet (home-assistant#5717)
* Fix add-on store repository getting removed without internet Currently, when a git command error happens in `pull()`, we declare the repository as corrupt. Subsequent system autofix runs then execute the reset resolution, which essentially removes the git repository from the system. In situations where the Internet fails right between the last Supervisor connectivity check and the add-on store repository update (the connectivity checks are throttled to once every 10 minutes while connectivity is considered good), or if the outage is only partial (e.g. reaching connectivity check works but the store repository is not reachable), this leads to a git command error which declares the repository as corrupt just as well, and ultimately leads to the removal of the add-on store repository from the local system. Run a git ls-remote first, which is used as an extra connectivity check. This will also avoid removing the repository if Internet connectivity works but the git provider is temporary down or not reachable. That said, it will also fail if the repository is no longer present. But this case needs extra handling anyways. * Run git ls-remote in executor
1 parent 324b059 commit 2602405

File tree

1 file changed

+7
-0
lines changed

1 file changed

+7
-0
lines changed

supervisor/store/git.py

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -132,6 +132,13 @@ async def pull(self) -> bool:
132132
async with self.lock:
133133
_LOGGER.info("Update add-on %s repository from %s", self.path, self.url)
134134

135+
try:
136+
git_cmd = git.Git()
137+
await self.sys_run_in_executor(git_cmd.ls_remote, "--heads", self.url)
138+
except git.CommandError as err:
139+
_LOGGER.warning("Wasn't able to update %s repo: %s.", self.url, err)
140+
raise StoreGitError() from err
141+
135142
try:
136143
branch = self.repo.active_branch.name
137144

0 commit comments

Comments
 (0)