Skip to content

Add option to run sshfs with idmap argument#31

Closed
MarekPikula wants to merge 1 commit into
lima-vm:masterfrom
MarekPikula:feature/sshfs-idmap-support
Closed

Add option to run sshfs with idmap argument#31
MarekPikula wants to merge 1 commit into
lima-vm:masterfrom
MarekPikula:feature/sshfs-idmap-support

Conversation

@MarekPikula

Copy link
Copy Markdown

It makes it possible to have user ID remapped between local and remote.

It makes it possible to have user ID remapped between local and remote.

Signed-off-by: Marek Pikuła <marek.pikula@embevity.com>
@MarekPikula MarekPikula force-pushed the feature/sshfs-idmap-support branch from bd19247 to 9941807 Compare May 18, 2022 14:53
Comment thread cmd/sshocker/run.go
Name: "sshfs-idmap",
Usage: "set sshfs idmap option",
Value: "none",
},

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'm feeling we should rather add generic -o to support arbitrary mount options.

Addition of sshfs-nonempty was design mistake and should be deprecated in favor of -o nonempty

@AkihiroSuda

Copy link
Copy Markdown
Member

Let me close this in favor of #51

Thanks anyway for your contribution.

@MarekPikula

Copy link
Copy Markdown
Author

Sure thing, I had a lot on my plate recently and didn't really get down to learning Go enough to make the change myself 😜 It's nice that it got merged 👍

@MarekPikula MarekPikula deleted the feature/sshfs-idmap-support branch October 16, 2022 16:15
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