From 3a8e22527357da1c5b701a22915288e398e7b93c Mon Sep 17 00:00:00 2001
From: "Yan, Zheng" <zyan@redhat.com>
Date: Mon, 17 Aug 2020 12:55:22 +0800
Subject: [PATCH] mds: don't track change of config
 'mds_replay_unsafe_with_closed_session'

the option is only used when mds is recovering.

Signed-off-by: "Yan, Zheng" <zyan@redhat.com>
---
 src/common/options.cc | 2 +-
 src/mds/MDSRank.cc    | 1 -
 src/mds/Server.cc     | 3 ---
 3 files changed, 1 insertion(+), 5 deletions(-)

diff --git a/src/common/options.cc b/src/common/options.cc
index 39d56304eac2..14426aaae982 100644
--- a/src/common/options.cc
+++ b/src/common/options.cc
@@ -7957,7 +7957,7 @@ std::vector<Option> get_mds_options() {
 
     Option("mds_replay_unsafe_with_closed_session", Option::TYPE_BOOL, Option::LEVEL_ADVANCED)
     .set_default(false)
-    .set_flag(Option::FLAG_RUNTIME)
+    .set_flag(Option::FLAG_STARTUP)
     .set_description("complete all the replay request when mds is restarted, no matter the session is closed or not"),
 
     Option("mds_default_dir_hash", Option::TYPE_INT, Option::LEVEL_ADVANCED)
diff --git a/src/mds/MDSRank.cc b/src/mds/MDSRank.cc
index e49ca9676143..c355ea175ca2 100644
--- a/src/mds/MDSRank.cc
+++ b/src/mds/MDSRank.cc
@@ -3621,7 +3621,6 @@ const char** MDSRankDispatcher::get_tracked_conf_keys() const
     "mds_recall_warning_decay_rate",
     "mds_request_load_average_decay_rate",
     "mds_session_cache_liveness_decay_rate",
-    "mds_replay_unsafe_with_closed_session",
     "mds_heartbeat_grace",
     NULL
   };
diff --git a/src/mds/Server.cc b/src/mds/Server.cc
index 3c7d8427f7a3..395a670bd647 100644
--- a/src/mds/Server.cc
+++ b/src/mds/Server.cc
@@ -1182,9 +1182,6 @@ void Server::handle_conf_change(const std::set<std::string>& changed) {
   if (changed.count("mds_forward_all_requests_to_auth")){
     forward_all_requests_to_auth = g_conf().get_val<bool>("mds_forward_all_requests_to_auth");
   }
-  if (changed.count("mds_replay_unsafe_with_closed_session")) {
-    replay_unsafe_with_closed_session = g_conf().get_val<bool>("mds_replay_unsafe_with_closed_session");
-  }
   if (changed.count("mds_cap_revoke_eviction_timeout")) {
     cap_revoke_eviction_timeout = g_conf().get_val<double>("mds_cap_revoke_eviction_timeout");
     dout(20) << __func__ << " cap revoke eviction timeout changed to "
-- 
2.47.3