Skip to content

Do not fail a successful install on a busy dmg - #40

Merged
mat007 merged 1 commit into
docker:mainfrom
ctalledo:fix-dmg-detach
Sep 11, 2026
Merged

mat007 merged 1 commit into
docker:mainfrom
ctalledo:fix-dmg-detach

Conversation

@ctalledo

@ctalledo ctalledo commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Installing on macOS fails with exit 16 when the volume is still busy right after install:

dmg installed
ls ./cache/desktopInstallSettings/
install-settings.json
hdiutil: couldn't unmount "disk6" - Resource busy
Error: Process completed with exit code 16

Desktop is installed and the settings have already been copied out by that point, so the run is lost to cleanup alone.

Retries the detach, then forces it, and warns rather than fails if it still will not go. A mount left behind is recoverable — a caller that mounts into a workspace path force unmounts and removes it before its next install, and on an ephemeral runner the machine is discarded anyway. Failing the step here is not.

Installing on macOS fails with exit 16 when the volume is still busy
right after install:

  dmg installed
  hdiutil: couldn't unmount "disk6" - Resource busy
  Error: Process completed with exit code 16

Desktop is installed and the settings have already been copied out by
that point, so the run is lost to cleanup alone. Something still holds
the volume open for a moment after the installer finishes.

Retry the detach, then force it, and warn rather than fail if it still
will not go. A mount left behind is recoverable: the caller that mounts
into a workspace path force unmounts and removes it before its next
install, and on an ephemeral runner the machine is discarded anyway.
Failing the step here is not recoverable.

Signed-off-by: Cesar Talledo <cesar.talledo@docker.com>
@mat007
mat007 merged commit 9b8f778 into docker:main Sep 11, 2026
2 checks passed
@ctalledo
ctalledo deleted the fix-dmg-detach branch September 11, 2026 18:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants