From: Sage Weil <sage@redhat.com>
Date: Sat, 14 Jul 2018 20:03:43 +0000 (-0500)
Subject: common/options: disable osd_skip_data_digest
X-Git-Tag: v14.0.1~848^2~1
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=d134c4985fc316a3bf71fa23d77613f16871a8e3;p=ceph.git

common/options: disable osd_skip_data_digest

This is flawed in that it stops doing digests based on the primary's
backend store, not the replicas.

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

diff --git a/src/common/options.cc b/src/common/options.cc
index 7046db836d9fb..c1e2714c07858 100644
--- a/src/common/options.cc
+++ b/src/common/options.cc
@@ -2386,7 +2386,7 @@ std::vector<Option> get_global_options() {
     .set_description(""),
 
     Option("osd_skip_data_digest", Option::TYPE_BOOL, Option::LEVEL_DEV)
-    .set_default(true)
+    .set_default(false)
     .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)