]> git.apps.os.sepia.ceph.com Git - ceph.git/commit
mds: allow all types of MDS caps
authorRishabh Dave <ridave@redhat.com>
Sun, 30 Apr 2023 13:01:00 +0000 (18:31 +0530)
committerRishabh Dave <ridave@redhat.com>
Fri, 14 Jul 2023 13:10:39 +0000 (18:40 +0530)
commit0c36929f258f2b32ded16bfaf40f99c27b668f44
treee0bba5710eaf009ebaf3b0aacd1260629d785ca2
parente40708108d4c765b528200fe13e3008c6be2b36f
mds: allow all types of MDS caps

MDS caps can contain 5 components: name of a CephFS, a path inside
CephFS, a flag for enabling root squashing mechanism, a UID and list of
GIDs. These 5 components result in 31 combinations, so there can be 31
types of MDS caps. Out of these, the current main branch only allows 11
combinations. This restriction is strange and inappropriate. Ideally,
all combinations should be allowed.

This strange restriction must've been created unintentionally by
previous developers while adding FS name and root squash to MDS caps. A
TODO for a allowing a subset of these combination was also left in
codebase:
https://github.com/ceph/ceph/blob/reef/src/mds/MDSAuthCaps.cc#L69

Fixes: https://tracker.ceph.com/issues/59388
Signed-off-by: Rishabh Dave <ridave@redhat.com>
src/mds/MDSAuthCaps.cc
src/mds/MDSAuthCaps.h