Support golang io/fs interface about go-nfs HOT 5 OPEN

Thanks for the feedback! (Sry for being slow)

Currently working in https://github.com/willscott/go-nfs/tree/feat/iofs

A bit confused and not too sure what's the best way to 'limit' how much of the file system they expose. I'm thinking of limiting it by using a wrapper. Something like how the change file attribute interface is only available by using certain wrappers in here

Also, I think if you pass in an os.DirFS without a wrapper it would be a read-only fs.

Looking into setting up a memfs and is currently choosing between wrapping a billy.fs or an afero.fs or both. billy.fs wrapper should be a better choice as it will provide backwards compatibility. Let me know your thoughts about this.

from go-nfs.

@willscott What's your thoughts about switching from billy to io/fs?

from go-nfs.

I'm happy to see this happen. I think the main thing is that my understanding is that there is not yet a clear standardization on what the extension of the core io/fs interface would be for mutation / changing. E.g. how the methods expressed in the Change link above should be conveyed back to the filesystem.

from go-nfs.

@willscott been playing around with io/fs and somewhat got a working fork https://github.com/scorpionknifes/go-nfs. Currently basing mutation by following os.DirFS and only the osnfs example works. Just want to get some feedback and thoughts about it.

I'm planning to get memfs working and write some tests (the fork is still in very early stages)

from go-nfs.

Cool!

I'll add you to this repo so you can work on a branch here and we can get this into a PR and merged as you get happy with it.

• You have each specific method (StatFS, CreateFS, etc) as a separate interface.

  • Can you re-use the upstream interface definitions where they exist, e.g. io/fs.StatFS?

• We should define an additional interface layer to be clear about what capabilities are needed for different types of mounts

  • (e.g. you don't need WriteFile if you support WriteAtFile as well, but you do need MkdirAll if you want a writable NFS mount)
  • I think there's probably a 'readable NFS mount' layer, and then 2 ways it can be extended - there's an additional set of interfaces to implement if you want to expose a 'read/write NFS mount', and a different set to expose a 'permissioned NFS mount'

• There should be some way for the user to 'limit' how much of the filesystem they expose: If i pass in a DirFS, it will currently be a read/write filesystem. It would be good to be able to specify somehow a bit more cleanly (right now i think it would be through a custom handler) that i want to set up the filesystem in read-only mode even if it has the writable methods on it.

from go-nfs.