From: Igor Fedotov <ifedotov@mirantis.com>
Date: Thu, 15 Sep 2016 13:44:00 +0000 (+0300)
Subject: os/bluestore: rename bluestore_compression option to bluestore_compression_mode.
X-Git-Tag: v11.1.0~539^2~2
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=6e28d6feb582de6a640d0d2b4e3c44e0426e1aac;p=ceph-ci.git

os/bluestore: rename bluestore_compression option to bluestore_compression_mode.

Signed-off-by: Igor Fedotov <ifedotov@mirantis.com>
---

diff --git a/src/common/config_opts.h b/src/common/config_opts.h
index a9248010f75..15db46407b5 100644
--- a/src/common/config_opts.h
+++ b/src/common/config_opts.h
@@ -977,7 +977,7 @@ OPTION(bluestore_min_alloc_size, OPT_U32, 0)
 OPTION(bluestore_min_alloc_size_hdd, OPT_U32, 64*1024)
 OPTION(bluestore_min_alloc_size_ssd, OPT_U32, 4*1024)
 OPTION(bluestore_max_alloc_size, OPT_U32, 0)
-OPTION(bluestore_compression, OPT_STR, "none")  // force|aggressive|passive|none
+OPTION(bluestore_compression_mode, OPT_STR, "none")  // force|aggressive|passive|none
 OPTION(bluestore_compression_algorithm, OPT_STR, "snappy")
 OPTION(bluestore_compression_min_blob_size, OPT_U32, 128*1024)
 OPTION(bluestore_compression_max_blob_size, OPT_U32, 512*1024)
diff --git a/src/os/bluestore/BlueStore.cc b/src/os/bluestore/BlueStore.cc
index 4cb5a0ccaa9..e0b564a4af4 100644
--- a/src/os/bluestore/BlueStore.cc
+++ b/src/os/bluestore/BlueStore.cc
@@ -2518,7 +2518,7 @@ const char **BlueStore::get_tracked_conf_keys() const
 {
   static const char* KEYS[] = {
     "bluestore_csum_type",
-    "bluestore_compression",
+    "bluestore_compression_mode",
     "bluestore_compression_algorithm",
     "bluestore_compression_min_blob_size",
     "bluestore_compression_max_blob_size",
@@ -2533,7 +2533,7 @@ void BlueStore::handle_conf_change(const struct md_config_t *conf,
   if (changed.count("bluestore_csum_type")) {
     _set_csum();
   }
-  if (changed.count("bluestore_compression") ||
+  if (changed.count("bluestore_compression_mode") ||
       changed.count("bluestore_compression_algorithm") ||
       changed.count("bluestore_compression_min_blob_size") ||
       changed.count("bluestore_compression_max_blob_size")) {
@@ -2546,13 +2546,13 @@ void BlueStore::_set_compression()
   comp_min_blob_size = g_conf->bluestore_compression_min_blob_size;
   comp_max_blob_size = g_conf->bluestore_compression_max_blob_size;
 
-  auto m = Compressor::get_comp_mode_type(g_conf->bluestore_compression);
+  auto m = Compressor::get_comp_mode_type(g_conf->bluestore_compression_mode);
   if (m) {
     comp_mode = *m;
   } else {
     derr << __func__ << " unrecognized value '"
-         << g_conf->bluestore_compression
-         << "' for bluestore_compression, reverting to 'none'"
+         << g_conf->bluestore_compression_mode
+         << "' for bluestore_compression_mode, reverting to 'none'"
          << dendl;
     comp_mode = Compressor::COMP_NONE;
   }
diff --git a/src/test/objectstore/store_test.cc b/src/test/objectstore/store_test.cc
index 2c17eb6ee4b..250c48f211c 100644
--- a/src/test/objectstore/store_test.cc
+++ b/src/test/objectstore/store_test.cc
@@ -915,19 +915,19 @@ TEST_P(StoreTest, CompressionTest) {
     return;
 
   g_conf->set_val("bluestore_compression_algorithm", "snappy");
-  g_conf->set_val("bluestore_compression", "force");
+  g_conf->set_val("bluestore_compression_mode", "force");
   g_ceph_context->_conf->apply_changes(NULL);
 
   doCompressionTest(store);
 
   g_conf->set_val("bluestore_compression_algorithm", "zlib");
-  g_conf->set_val("bluestore_compression", "force");
+  g_conf->set_val("bluestore_compression_mode", "force");
   g_ceph_context->_conf->apply_changes(NULL);
 
   doCompressionTest(store);
 
   g_conf->set_val("bluestore_compression_algorithm", "snappy");
-  g_conf->set_val("bluestore_compression", "none");
+  g_conf->set_val("bluestore_compression_mode", "none");
   g_ceph_context->_conf->apply_changes(NULL);
 }
 
@@ -948,8 +948,8 @@ TEST_P(StoreTest, garbageCollection) {
       r = apply_transaction(store, &osr, std::move(t));\
       ASSERT_EQ(r, 0);\
   }
-  g_conf->set_val("bluestore_compression", "none");
-  //g_conf->set_val("bluestore_compression", "force");
+  g_conf->set_val("bluestore_compression_mode", "none");
+  //g_conf->set_val("bluestore_compression_mode", "force");
   g_conf->set_val("bluestore_merge_gc_data", "true"); 
   g_ceph_context->_conf->apply_changes(NULL);
 
@@ -1100,7 +1100,7 @@ TEST_P(StoreTest, garbageCollection) {
       ASSERT_EQ(r, 0);
      }
   }
-  g_conf->set_val("bluestore_compression", "none");
+  g_conf->set_val("bluestore_compression_mode", "none");
   g_ceph_context->_conf->apply_changes(NULL);
 }
 
@@ -1323,7 +1323,7 @@ TEST_P(StoreTest, SimpleObjectTest) {
 TEST_P(StoreTest, BluestoreStatFSTest) {
   if(string(GetParam()) != "bluestore")
     return;
-  g_conf->set_val("bluestore_compression", "force");
+  g_conf->set_val("bluestore_compression_mode", "force");
   g_conf->set_val("bluestore_min_alloc_size", "65536");
   g_ceph_context->_conf->apply_changes(NULL);
   int r = store->umount();
@@ -1563,7 +1563,7 @@ TEST_P(StoreTest, BluestoreStatFSTest) {
     ASSERT_EQ( 0u, statfs.compressed);
     ASSERT_EQ( 0u, statfs.compressed_allocated);
   }
-  g_conf->set_val("bluestore_compression", "none");
+  g_conf->set_val("bluestore_compression_mode", "none");
   g_conf->set_val("bluestore_min_alloc_size", "0");
   g_ceph_context->_conf->apply_changes(NULL);
 }
@@ -4459,7 +4459,7 @@ TEST_P(StoreTest, SyntheticMatrixCsumVsCompression) {
     { "max_size", "262144", 0 },
     { "alignment", "512", 0 },
     { "bluestore_min_alloc_size", "4096", "16384", 0 },
-    { "bluestore_compression", "force", 0},
+    { "bluestore_compression_mode", "force", 0},
     { "bluestore_compression_algorithm", "snappy", "zlib", 0 },
     { "bluestore_csum_type", "crc32c", 0 },
     { "bluestore_default_buffered_read", "true", "false", 0 },
@@ -4478,7 +4478,7 @@ TEST_P(StoreTest, SyntheticMatrixCompression) {
     { "max_size", "4194304", 0 },
     { "alignment", "65536", 0 },
     { "bluestore_min_alloc_size", "4096", "65536", 0 },
-    { "bluestore_compression", "force", "aggressive", "passive", "none", 0},
+    { "bluestore_compression_mode", "force", "aggressive", "passive", "none", 0},
     { "bluestore_default_buffered_write", "false", 0 },
     { 0 },
   };
@@ -4494,7 +4494,7 @@ TEST_P(StoreTest, SyntheticMatrixCompressionAlgorithm) {
     { "max_size", "4194304", 0 },
     { "alignment", "65536", 0 },
     { "bluestore_compression_algorithm", "zlib", "snappy", 0 },
-    { "bluestore_compression", "force", 0 },
+    { "bluestore_compression_mode", "force", 0 },
     { "bluestore_default_buffered_write", "false", 0 },
     { 0 },
   };
@@ -4511,7 +4511,7 @@ TEST_P(StoreTest, SyntheticMatrixNoCsum) {
     { "alignment", "512", 0 },
     { "bluestore_min_alloc_size", "65536", "4096", 0 },
     { "bluestore_max_blob_size", "262144", 0 },
-    { "bluestore_compression", "force", "none", 0},
+    { "bluestore_compression_mode", "force", "none", 0},
     { "bluestore_csum_type", "none", 0},
     { "bluestore_default_buffered_read", "true", "false", 0 },
     { "bluestore_default_buffered_write", "true", 0 },