feat(transmission): revert to VirtioFS with deferred cleanup#102
Merged
smartwatermelon merged 8 commits intomainfrom Mar 29, 2026
Merged
feat(transmission): revert to VirtioFS with deferred cleanup#102smartwatermelon merged 8 commits intomainfrom
smartwatermelon merged 8 commits intomainfrom
Conversation
Replace VM-internal NFS mount with VirtioFS pass-through from the host NFS mount at ~/.local/mnt/DSMedia. Apple's vzNAT keeps dropping TCP connectivity to the NAS, making the VM-internal NFS mount unreliable. Removes Section 2b (VM-internal NFS mount setup via systemd) and Section 2c (NFS watchdog timer) since both are unnecessary when the host NFS mount is passed through via VirtioFS. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Update the podman-machine-start.sh template to use the VirtioFS pass-through path instead of the VM-internal NFS mount. Removes the pre-start NFS mount check (no longer needed with VirtioFS) and uses NFS_MOUNT_POINT variable for the data volume path. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Change delete-local-data from true to false in torrent removal. This lets Transmission close its FDs without triggering .nfs.* silly-renames through VirtioFS. Actual cleanup of pending-move/ is handled by a separate periodic script that verifies each directory is no longer tracked by Transmission before deleting. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Hourly sweep of pending-move/ that only removes directories confirmed absent from Transmission's torrent list via RPC. Prevents deletion of un-processed torrents, failed FileBot runs, and non-media files (ISOs). Companion to the delete-local-data:false change — separates 'release the lock' (torrent removal) from 'delete the files' (this script).
Adds pending-move-cleanup.sh deployment and an hourly LaunchAgent to podman-transmission-setup.sh. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
![sentry[bot]](https://avatars.githubusercontent.com/in/12637?s=60&v=4){kind=small}
1. Replace hardcoded DSMedia with __NAS_SHARE_NAME__ placeholder (MEDIUM: script would silently skip cleanup for non-default share names) 2. Replace grep/sed JSON parsing with python3 json module (HIGH: regex couldn't handle escaped quotes in torrent names, potentially causing false "not tracked" matches and data deletion) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Replace hardcoded DSMedia in MACOS_NAS_PREFIX with __NAS_SHARE_NAME__ placeholder, and add sed substitution in setup script deployment. Addresses Seer finding on PR #102. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Replace hardcoded DSMedia with ${NAS_SHARE_NAME} in the confirmation
prompt and NAS bind mount validation section.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
delete-local-datatofalsein trigger watcher torrent removal to avoid.nfs.*silly-rename files through VirtioFSpending-move-cleanup.sh— hourly sweep that only removes directories confirmed absent from Transmission's torrent list via RPC (safe for un-processed torrents, failed FileBot, ISOs)Context
VM-internal NFS mount (implemented 2026-03-16) was unreliable due to Apple vzNAT losing TCP forwarding to LAN hosts while ICMP continued working. This caused the NFS mount inside the Podman VM to drop, triggering Permission denied errors. Occurred 3 times in one session on 2026-03-28.
The host-side NFS mount (managed by the existing macOS watchdog) is rock-solid. VirtioFS passthrough reintroduces
.nfs.*silly-rename files, mitigated by separating "release the lock" (torrent removal without file deletion) from "delete the files" (hourly cleanup script).Test plan
AI review: code-reviewer + adversarial-reviewer (1 clean iteration, pre-commit + pre-push)
🤖 Generated with Claude Code