]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
mds/MDSAuthCaps: re-word an error message for better clarity
authorRishabh Dave <ridave@redhat.com>
Thu, 28 Sep 2023 14:29:24 +0000 (19:59 +0530)
committerRishabh Dave <ridave@redhat.com>
Fri, 6 Oct 2023 08:52:14 +0000 (14:22 +0530)
Signed-off-by: Rishabh Dave <ridave@redhat.com>
qa/tasks/cephfs/test_admin.py
src/mds/MDSAuthCaps.cc

index 4a18faa273178d5ac4e853e996d76378c5142c14..923871f88e3ac636c4931b6c634aacffc6e6c2de 100644 (file)
@@ -1806,8 +1806,8 @@ class TestPermErrMsg(CephFSTestCase):
     FS1_NAME, FS2_NAME, FS3_NAME = 'abcd', 'efgh', 'ijkl'
 
     EXPECTED_ERRNO = 22
-    EXPECTED_ERRMSG = ("Permission flags in MDS caps must start with 'r' or "
-                       "'rw' or be '*' or 'all'")
+    EXPECTED_ERRMSG = ("Permission flags in MDS capability string must be '*' "
+                       "or 'all' or must start with 'r'")
 
     MONCAP = f'allow r fsname={FS1_NAME}'
     OSDCAP = f'allow rw tag cephfs data={FS1_NAME}'
index df415f17d626dde94455915695fded576221e54f..7ddad41a08fa217447fc4fe13da1b7c7948648b4 100644 (file)
@@ -356,8 +356,8 @@ bool MDSAuthCaps::parse(string_view str, ostream *err)
 
     if (err) {
       if (string(iter, end).find("allow") != string::npos) {
-       *err << "Permission flags in MDS caps must start with 'r' or " <<
-              "'rw' or be '*' or 'all'";
+       *err << "Permission flags in MDS capability string must be '*' or "
+           << "'all' or must start with 'r'";
       } else {
        *err << "mds capability parse failed, stopped at '"
             << string(iter, end) << "' of '" << str << "'";