mountpoint-s3 icon indicating copy to clipboard operation
mountpoint-s3 copied to clipboard

Startup option to ignore mode change errors

Open pawalt opened this issue 1 year ago • 5 comments

Tell us more about this new feature.

We currently use mountpoint-s3 to allow our users to mount buckets in their containers as regular volumes. This has been working well, but there's one footgun that some of our users have run into - the mode changing errors. We have some users with pre-written scripts that automatically do a chmod on their files even if the mode is already properly set. For these users, mountpoint breaks their scripts as it throws errors on the chmod.

Would folks be open to a change to allow mountpoint to ignore change mode errors, effectively making them a no-op? This would be opt-in startup behavior, and it would make our lives much easier.

pawalt avatar Aug 08 '24 14:08 pawalt

Hey @pawalt, thanks for opening this feature request. I'll bring this to the team, I can see why this could be useful.

dannycjones avatar Aug 12 '24 17:08 dannycjones

FYI we have implemented this on our fork: https://github.com/modal-labs/mountpoint-s3/pull/3

pawalt avatar Aug 15 '24 17:08 pawalt

hey @dannycjones any updates on prioritizing this? It would be useful for us as well if we want to replace the Hadoop fs with s3 mountpoint.

ssejal1508 avatar Oct 18 '24 21:10 ssejal1508

Hey @helenweng-stripe, thanks for sharing your use-case. There is no update we can share as of now. I'd suggest adding thumps up reaction (👍) to the main comment to help us to track interest on this feature.

unexge avatar Oct 21 '24 10:10 unexge

Hello, this also impacts django applications because the default storage backend health check runs chmod on a temp file https://github.com/django/django/blob/5a1cae3a5675c5733daf5949759476d65aa0e636/django/core/files/storage/filesystem.py#L141

daveshm avatar Feb 26 '25 03:02 daveshm