client: move a client's option to mds-client.yaml

mds_max_retries_on_remount_failure option is used by Client.cc only.

Fixes: https://tracker.ceph.com/issues/56532
Signed-off-by: Xiubo Li <xiubli@redhat.com>

diff --git a/PendingReleaseNotes b/PendingReleaseNotes
index b6a18ff9a997314028c8f3f7667e5c45de47c252..97e47148abb82a8a550ef2e0819f4b691946c6c2 100644 (file)
--- a/PendingReleaseNotes
+++ b/PendingReleaseNotes
@@ -100,6 +100,10 @@
   potentially different from that of the parent image.  The efficient
   copy-on-write semantics intrinsic to unformatted (regular) cloned images
   are retained.
+* CEPHFS: Rename the `mds_max_retries_on_remount_failure` option to
+  `client_max_retries_on_remount_failure` and move it from mds.yaml.in to
+  mds-client.yaml.in because this option was only used by MDS client from its
+  birth.
 
 >=17.2.1
 

diff --git a/src/client/Client.cc b/src/client/Client.cc
index 4c180ef754da4065a308e458119b3b6c33431292..428fe83526a747031a1d1c185f142e2dccdb31d6 100644 (file)
--- a/src/client/Client.cc
+++ b/src/client/Client.cc
@@ -4564,7 +4564,7 @@ void Client::remove_session_caps(MetaSession *s, int err)
 
 std::pair<int, bool> Client::_do_remount(bool retry_on_error)
 {
-  uint64_t max_retries = cct->_conf.get_val<uint64_t>("mds_max_retries_on_remount_failure");
+  uint64_t max_retries = cct->_conf.get_val<uint64_t>("client_max_retries_on_remount_failure");
   bool abort_on_failure = false;
 
   errno = 0;

diff --git a/src/common/options/mds-client.yaml.in b/src/common/options/mds-client.yaml.in
index 7f96fa30d24431a8d710841703585cf70813550d..4e599d4cfa8a89138d6b4adc36a15605465319a2 100644 (file)
--- a/src/common/options/mds-client.yaml.in
+++ b/src/common/options/mds-client.yaml.in
@@ -453,6 +453,14 @@ options:
   services:
   - mds_client
   with_legacy: true
+- name: client_max_retries_on_remount_failure
+  type: uint
+  level: advanced
+  desc: number of consecutive failed remount attempts for invalidating kernel dcache
+    after which client would abort.
+  default: 5
+  services:
+  - mds_client
 - name: client_die_on_failed_remount
   type: bool
   level: dev

diff --git a/src/common/options/mds.yaml.in b/src/common/options/mds.yaml.in
index f22d867cfea258be34bb71a11d455e4d4ee54696..8d54b851a518075a73008bf0ab0cab49ec44ba12 100644 (file)
--- a/src/common/options/mds.yaml.in
+++ b/src/common/options/mds.yaml.in
@@ -1343,14 +1343,6 @@ options:
   default: 0
   services:
   - mds
-- name: mds_max_retries_on_remount_failure
-  type: uint
-  level: advanced
-  desc: number of consecutive failed remount attempts for invalidating kernel dcache
-    after which client would abort.
-  default: 5
-  services:
-  - mds
 - name: mds_dump_cache_threshold_formatter
   type: size
   level: dev