mds/MDSAuthCaps: re-word an error message for better clarity

Signed-off-by: Rishabh Dave <ridave@redhat.com>

diff --git a/qa/tasks/cephfs/test_admin.py b/qa/tasks/cephfs/test_admin.py
index 4a18faa273178d5ac4e853e996d76378c5142c14..923871f88e3ac636c4931b6c634aacffc6e6c2de 100644 (file)
--- a/qa/tasks/cephfs/test_admin.py
+++ b/qa/tasks/cephfs/test_admin.py
@@ -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}'

diff --git a/src/mds/MDSAuthCaps.cc b/src/mds/MDSAuthCaps.cc
index df415f17d626dde94455915695fded576221e54f..7ddad41a08fa217447fc4fe13da1b7c7948648b4 100644 (file)
--- a/src/mds/MDSAuthCaps.cc
+++ b/src/mds/MDSAuthCaps.cc
@@ -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 << "'";