Skip to content

fix: improve mount retry logic and import error handling - #136

Merged
javi11 merged 1 commit into
mainfrom
fix/mount-retry-and-import-error-handling
Dec 16, 2025
Merged

fix: improve mount retry logic and import error handling#136
javi11 merged 1 commit into
mainfrom
fix/mount-retry-and-import-error-handling

Conversation

@javi11

@javi11 javi11 commented Dec 16, 2025

Copy link
Copy Markdown
Collaborator

Summary

  • Always create new mount instance with fresh config values instead of reusing stale configuration when remounting
  • Add RC unmount and force unmount cleanup before mount retries to prevent stale mount state from blocking retries
  • Add macOS-specific unmount commands (umount -f, diskutil unmount force) alongside existing Linux fusermount commands
  • Return errors from refresh and STRM creation instead of silently logging, ensuring import failures are properly propagated

Test plan

  • Test mount/remount operations on macOS
  • Test mount/remount operations on Linux
  • Verify import queue properly fails when refresh or STRM creation fails
  • Test mount retry behavior after failed mount attempts

🤖 Generated with Claude Code

- Always create new mount instance with fresh config values instead of
  reusing stale configuration when remounting
- Add RC unmount and force unmount cleanup before mount retries to
  prevent stale mount state from blocking retries
- Add macOS-specific unmount commands (umount -f, diskutil unmount force)
  alongside existing Linux fusermount commands
- Return errors from refresh and STRM creation instead of silently
  logging, ensuring import failures are properly propagated

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@javi11
javi11 merged commit 58bc343 into main Dec 16, 2025
1 check passed
@javi11
javi11 deleted the fix/mount-retry-and-import-error-handling branch February 4, 2026 13:41
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.

1 participant