From: Joao Eduardo Luis <joao@suse.de>
Date: Mon, 4 Dec 2017 23:38:31 +0000 (+0000)
Subject: mon/OSDMonitor: allow blocking trim if option is enabled
X-Git-Tag: v13.1.0~393^2~7
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=029807b3e97962483fd123fb64c1aa5ff2adfccb;p=ceph.git

mon/OSDMonitor: allow blocking trim if option is enabled

We need to allow blocking osdmap trimming to test osdmap pruning.

Signed-off-by: Joao Eduardo Luis <joao@suse.de>
---

diff --git a/src/common/options.cc b/src/common/options.cc
index c9ddfa9c7121..bd1591efacf5 100644
--- a/src/common/options.cc
+++ b/src/common/options.cc
@@ -1616,6 +1616,14 @@ std::vector<Option> get_global_options() {
         "to run on production systems, or would only be relevant while "
         "testing or debugging."),
 
+    Option("mon_debug_block_osdmap_trim", Option::TYPE_BOOL, Option::LEVEL_DEV)
+    .set_default(false)
+    .set_description("Block OSDMap trimming while the option is enabled.")
+    .set_long_description(
+        "Blocking OSDMap trimming may be quite helpful to easily reproduce "
+        "states in which the monitor keeps (hundreds of) thousands of "
+        "osdmaps."),
+
     Option("mon_debug_deprecated_as_obsolete", Option::TYPE_BOOL, Option::LEVEL_DEV)
     .set_default(false)
     .set_description(""),
diff --git a/src/mon/Monitor.cc b/src/mon/Monitor.cc
index 0297addc545d..d25744bcc02d 100644
--- a/src/mon/Monitor.cc
+++ b/src/mon/Monitor.cc
@@ -456,6 +456,7 @@ const char** Monitor::get_tracked_conf_keys() const
     "mon_osdmap_full_prune_txsize",
     // debug options - observed, not handled
     "mon_debug_extra_checks",
+    "mon_debug_block_osdmap_trim",
     NULL
   };
   return KEYS;
diff --git a/src/mon/OSDMonitor.cc b/src/mon/OSDMonitor.cc
index 7ef7301ea183..4e8f08d57d42 100644
--- a/src/mon/OSDMonitor.cc
+++ b/src/mon/OSDMonitor.cc
@@ -1510,6 +1510,15 @@ version_t OSDMonitor::get_trim_to() const
       return 0;
     }
   }
+
+  if (g_conf->get_val<bool>("mon_debug_block_osdmap_trim")) {
+    dout(0) << __func__
+            << " blocking osdmap trim"
+               " ('mon_debug_block_osdmap_trim' set to 'true')"
+            << dendl;
+    return 0;
+  }
+
   {
     epoch_t floor = get_min_last_epoch_clean();
     dout(10) << " min_last_epoch_clean " << floor << dendl;