[LU-2738] add an MDT /proc tunable to allow non-root users to do lfs mkdir and lfs rm_entry

• Status: Resolved
• Project: Lustre
• Affects Version/s: Lustre 2.4.0
• Fix Version/s: Lustre 2.4.0
• Type: Bug
• Priority: Blocker
• Reporter: Di Wang
• Assignee: Di Wang
• Resolution: Fixed
• Labels: MB
• Severity: 3
• Epic: MB

Description

Add an MDT /proc tunable to allow non-root users to do lfs mkdir and lfs rm_entry

Comments

[Robert Read (Inactive)]

I would think this would also be a conf_param so it's persistent. Perhaps that's what you meant?

[Di Wang]

Oh, we probably want this to be enable/disable in real time, in stead of storing in config log, IMHO.

And also, currently, lfs mkdir, rm_entry should only happen on MDT0, except you enable another /proc parameter to enable this on other MDT. So I think administrator can do this by set_param on MDT0 directly. And later when we enable this(create remote directory) to the normal user, this will be the default setting for all MDT, i.e. this can not be disabled.
so maybe we should not add such this temporary thing to the permanent conf log, IMHO.

[Robert Read (Inactive)]

I see, that makes sense.

[Andreas Dilger]

Note that any parameter that is a /proc/fs/lustre/

{obd_type}

tunable should also be a conf_param tunable by default, so there shouldn't be a need to make a distinction here.

[John Hammond]

Since 'lfs rm_entry' can create orphaned directories I don't believe that this is something worth considering for non-root users. (This sort of thing is why systems have admins). If you do go ahead and make it a tunable then please make it a separate tunable from the 'lfs mkdir' tunable.

Also consider making it (or both) a gid rather than binary tunable, with say (gid_t) -1 meaning that any user is capable of lfs mkdir.

[Di Wang]

http://review.whamcloud.com/5442

[Peter Jones]

Landed for 2.4