Re: [PATCH] fs/ceph/super: add mount options "snapdir{mode,uid,gid}"

From: Max Kellermann
Date: Sun Oct 09 2022 - 02:49:47 EST


On Sun, Oct 9, 2022 at 8:23 AM Xiubo Li <xiubli@xxxxxxxxxx> wrote:
> I don't think this is a good place to implement this in client side.
> Should this be a feature in cephx.

What's cephx? "git grep cephx" didn't reveal anything that looked useful to me.

> With this for the same directories in different mounts will behave
> differently. Isn't that odd ?

Just like different mounts can have different snapdir names currently.
That's just as odd, and I tried to imitate what's already there.

I don't have an opinion on how this should be implemented, all I want
is restrict who gets access to cephfs snapshots. Please explain how
you want it, and I'll send an amended patch.