Skip to content

Add NFS Filesystem Provider#3276

Open
OzGav wants to merge 3 commits intodevfrom
nfs-filesystem-provider
Open

Add NFS Filesystem Provider#3276
OzGav wants to merge 3 commits intodevfrom
nfs-filesystem-provider

Conversation

@OzGav
Copy link
Contributor

@OzGav OzGav commented Mar 2, 2026

No description provided.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 2, 2026

🔒 Dependency Security Report

✅ No dependency changes detected in this PR.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a new filesystem provider that mounts an NFS export locally and then reuses the existing LocalFileSystemProvider logic to index/browse/play media from the mounted path.

Changes:

  • Introduces filesystem_nfs provider module with config flow and mount/unmount implementation.
  • Adds NFS provider manifest and icons.
  • Expands filesystem local ignore-directory list (e.g., adds @eaDir) and reformats it.

Reviewed changes

Copilot reviewed 5 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
music_assistant/providers/filesystem_nfs/provider.py Implements NFS mount/unmount wrapper around LocalFileSystemProvider.
music_assistant/providers/filesystem_nfs/init.py Provider setup + config entries (host/export path/subfolder/NFS version).
music_assistant/providers/filesystem_nfs/constants.py Defines config keys used by the NFS provider.
music_assistant/providers/filesystem_nfs/manifest.json Registers the new provider (domain/type/stage/docs/icon).
music_assistant/providers/filesystem_nfs/icon.svg Provider icon asset.
music_assistant/providers/filesystem_nfs/icon_monochrome.svg Provider monochrome icon asset.
music_assistant/providers/filesystem_local/helpers.py Updates IGNORE_DIRS to include additional ignored directories.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants