From: Sage Weil <sage@redhat.com>
Date: Sat, 14 Jul 2018 18:26:31 +0000 (-0500)
Subject: common/options: make osd_skip_data_digest a DEV option
X-Git-Tag: v13.2.1~25^2~2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=35ed68a8b239c69c879a065772e3a7df289b4391;p=ceph.git

common/options: make osd_skip_data_digest a DEV option

And document/warn to never turn it off after it's been on.

Signed-off-by: Sage Weil <sage@redhat.com>
(cherry picked from commit c854ed9b33a5a293c8ace15718e26f74237e3332)
---

diff --git a/src/common/options.cc b/src/common/options.cc
index 7983a5d3467..7a8b591e3c5 100644
--- a/src/common/options.cc
+++ b/src/common/options.cc
@@ -2316,9 +2316,9 @@ std::vector<Option> get_global_options() {
     .set_flag(Option::FLAG_STARTUP)
     .set_description(""),
 
-    Option("osd_skip_data_digest", Option::TYPE_BOOL, Option::LEVEL_ADVANCED)
+    Option("osd_skip_data_digest", Option::TYPE_BOOL, Option::LEVEL_DEV)
     .set_default(true)
-    .set_description(""),
+    .set_description("Do not store full-object checksums if the backend (bluestore) does its own checksums.  Do not ever turn this off if it has ever been turned on."),
 
     Option("osd_op_queue", Option::TYPE_STR, Option::LEVEL_ADVANCED)
     .set_default("wpq")