]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
osd/OSDCap: fix misleading grammar comments 61602/head
authorIlya Dryomov <idryomov@gmail.com>
Mon, 27 Jan 2025 11:29:54 +0000 (12:29 +0100)
committerIlya Dryomov <idryomov@gmail.com>
Fri, 31 Jan 2025 09:15:08 +0000 (10:15 +0100)
The restrictions on pool name and namespace have been independent of
each other for ages.  Specifying namespace[=]<namespace> doesn't require
specifying pool[=]<pool> like is currently suggested -- neither for
regular "allow" grants nor for "profile" grants.

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
(cherry picked from commit d99367e1e03cbeefd33d7dd9405245e089c2237c)

src/osd/OSDCap.cc

index 1a5348ed65125ba3f511cd0d93d66c0ee6fe004f..f8c59d2ef87778d1fd0ae2478bdb04625df1e99b 100644 (file)
@@ -440,7 +440,7 @@ struct OSDCapParser : qi::grammar<Iterator, OSDCap()>
               >> (lit('=') | spaces)
               >> estr >> -char_('*'));
 
-    // match := [pool[=]<poolname> [namespace[=]<namespace>]] [object_prefix <prefix>]
+    // match := [pool[=]<poolname>] [namespace[=]<namespace>] [object_prefix <prefix>]
     object_prefix %= -(spaces >> lit("object_prefix") >> spaces >> str);
     pooltag %= (spaces >> lit("tag")
                >> spaces >> str // application
@@ -473,7 +473,7 @@ struct OSDCapParser : qi::grammar<Iterator, OSDCap()>
       (rwxa)                      [_val = phoenix::construct<OSDCapSpec>(_1)] |
       (class_name >> method_name) [_val = phoenix::construct<OSDCapSpec>(_1, _2)]);
 
-    // profile := profile <name> [pool[=]<pool> [namespace[=]<namespace>]]
+    // profile := profile <name> [pool[=]<pool>] [namespace[=]<namespace>]
     profile_name %= (lit("profile") >> (lit('=') | spaces) >> str);
     profile = (
       (profile_name >> pool_name >> nspace) [_val = phoenix::construct<OSDCapProfile>(_1, _2, _3)] |