Skip to content

Commit 9aa6665

Browse files
committed
Clarify outer DockerError comment in App.load()
The comment claimed "a future load will reattempt and surface a MISSING_IMAGE repair if appropriate", but App.load() is only called at Supervisor startup, on fresh install, and on backup restore — there is no automatic retry mechanism. Reword to match reality: the CRITICAL log captures the issue for diagnostics (Sentry), and the user can trigger a manual repair once the daemon is healthy.
1 parent 6e55d18 commit 9aa6665

1 file changed

Lines changed: 4 additions & 3 deletions

File tree

supervisor/apps/app.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -275,9 +275,10 @@ async def load(self) -> None:
275275
self._create_missing_image_issue()
276276
except DockerError as err:
277277
# Docker error other than a clean "image not found" - we can't
278-
# tell whether the image is actually missing. Log and leave the
279-
# addon detached; a future load will reattempt and surface a
280-
# MISSING_IMAGE repair if appropriate.
278+
# tell whether the image is actually missing. Log so the issue
279+
# is visible (CRITICAL is captured by the Sentry integration)
280+
# and leave the app detached; the user can attempt a manual
281+
# repair from the UI once the underlying cause is resolved.
281282
_LOGGER.critical(
282283
"Docker error loading app %s, leaving detached: %s", self.slug, err
283284
)

0 commit comments

Comments
 (0)