]> git.apps.os.sepia.ceph.com Git - ceph.git/commit
rbd: make common options override krbd-specific options 37408/head
authorIlya Dryomov <idryomov@gmail.com>
Fri, 25 Sep 2020 07:56:29 +0000 (09:56 +0200)
committerIlya Dryomov <idryomov@gmail.com>
Fri, 25 Sep 2020 07:56:29 +0000 (09:56 +0200)
commit07a2580cd35e096953e4b16d6840127237e66886
tree198fb1569cb9888c7f5488739d55a0a54b3d593e
parentefdb0db2de7b1795bace506fc1538c0325fdcbbc
rbd: make common options override krbd-specific options

ceph-csi has added support for passing custom map and unmap options via
mapOptions and unmapOptions storage class parameters.  However, it also
uses --read-only for implementing ROX (ReadOnlyMany) PVs.  If the user
supplies "mapOptions: rw", they will get around the intended read-only
restriction (at least on the block device).

ceph-csi could be patched to use "-o ro", but it actually makes sense
for common options to win over device type-specific equivalents.

Fixes: https://tracker.ceph.com/issues/47625
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
(cherry picked from commit a107c47360ecdb8c09768ca9eab2341100245711)

Conflicts:
src/tools/rbd/action/Kernel.cc [ snapshot quiesce support and
  commit 34f539d8af33 ("rbd: delay parsing of default kernel map
  options") not in octopus ]
doc/man/8/rbd.rst
qa/rbd/krbd_blkroset.t
src/tools/rbd/action/Kernel.cc