]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
cleanup: Remove debug option osd_debug_scrub_chance_rewrite_digest
authorDavid Zafman <dzafman@redhat.com>
Wed, 30 May 2018 18:47:04 +0000 (11:47 -0700)
committerDavid Zafman <dzafman@redhat.com>
Thu, 31 May 2018 00:07:52 +0000 (17:07 -0700)
This option seems pointless and there are no test cases that use it.

Signed-off-by: David Zafman <dzafman@redhat.com>
src/common/legacy_config_opts.h
src/common/options.cc
src/osd/PGBackend.cc

index cdaf6f1f8818a3a36fc8e4eabf15b8ae6636c297..2f1ca3fd4bdbd3da99828768690f859c543bb1c0 100644 (file)
@@ -772,7 +772,6 @@ OPTION(osd_debug_drop_ping_probability, OPT_DOUBLE)
 OPTION(osd_debug_drop_ping_duration, OPT_INT)
 OPTION(osd_debug_op_order, OPT_BOOL)
 OPTION(osd_debug_verify_missing_on_start, OPT_BOOL)
-OPTION(osd_debug_scrub_chance_rewrite_digest, OPT_U64)
 OPTION(osd_debug_verify_snaps, OPT_BOOL)
 OPTION(osd_debug_verify_stray_on_activate, OPT_BOOL)
 OPTION(osd_debug_skip_full_check_in_backfill_reservation, OPT_BOOL)
index 277570f5b29557e2e3b14c9560821dbb4180e821..0dd966d572014494ade1c7fe2be2df099a03aa38 100644 (file)
@@ -3243,10 +3243,6 @@ std::vector<Option> get_global_options() {
     .set_default(false)
     .set_description(""),
 
-    Option("osd_debug_scrub_chance_rewrite_digest", Option::TYPE_UINT, Option::LEVEL_DEV)
-    .set_default(0)
-    .set_description(""),
-
     Option("osd_debug_verify_snaps", Option::TYPE_BOOL, Option::LEVEL_DEV)
     .set_default(false)
     .set_description(""),
index db823d19a524d0de132196cac1206957c07e2a2d..260ad4253bc4a69955d6cd8edeba80cb56bb94fe 100644 (file)
@@ -1063,13 +1063,6 @@ void PGBackend::be_compare_scrubmaps(
        dout(20) << __func__ << " missing digest on " << *k << dendl;
        update = MAYBE;
       }
-      if (auth_object.digest_present && auth_object.omap_digest_present &&
-         cct->_conf->osd_debug_scrub_chance_rewrite_digest &&
-         (((unsigned)rand() % 100) >
-          cct->_conf->osd_debug_scrub_chance_rewrite_digest)) {
-       dout(20) << __func__ << " randomly updating digest on " << *k << dendl;
-       update = MAYBE;
-      }
 
       // recorded digest != actual digest?
       if (auth_oi.is_data_digest() && auth_object.digest_present &&