From 029807b3e97962483fd123fb64c1aa5ff2adfccb Mon Sep 17 00:00:00 2001
From: Joao Eduardo Luis <joao@suse.de>
Date: Mon, 4 Dec 2017 23:38:31 +0000
Subject: [PATCH] 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>
---
 src/common/options.cc | 8 ++++++++
 src/mon/Monitor.cc    | 1 +
 src/mon/OSDMonitor.cc | 9 +++++++++
 3 files changed, 18 insertions(+)

diff --git a/src/common/options.cc b/src/common/options.cc
index c9ddfa9c7121d..bd1591efacf50 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 0297addc545d9..d25744bcc02de 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 7ef7301ea1830..4e8f08d57d420 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;
-- 
2.39.5