From: Adam C. Emerson <aemerson@redhat.com>
Date: Wed, 24 Mar 2021 19:47:42 +0000 (-0400)
Subject: rgw: Remove spurious `num_rados_handles` option
X-Git-Tag: v17.1.0~2489^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=4bc2a30c24ccae3c3f1d64cae5d97dd51ce5ccb2;p=ceph.git

rgw: Remove spurious `num_rados_handles` option

While any code that used the option is gone, it appears the option
itself was accidentally resurrected in various options.cc refactors
and transitions.

Signed-off-by: Adam C. Emerson <aemerson@redhat.com>
---

diff --git a/src/common/options.cc b/src/common/options.cc
index 8003b4bdcb606..39827e7ffda0a 100644
--- a/src/common/options.cc
+++ b/src/common/options.cc
@@ -6188,14 +6188,6 @@ std::vector<Option> get_rgw_options() {
         "better concurrency of these messages, at the cost of more resource "
         "utilization."),
 
-    Option("rgw_num_rados_handles", Option::TYPE_UINT, Option::LEVEL_ADVANCED)
-    .set_default(1)
-    .set_description("Number of librados handles that RGW uses.")
-    .set_long_description(
-        "This param affects the number of separate librados handles it uses to "
-        "connect to the RADOS backend, which directly affects the number of connections "
-        "RGW will have to each OSD. A higher number affects resource utilization."),
-
     Option("rgw_verify_ssl", Option::TYPE_BOOL, Option::LEVEL_ADVANCED)
     .set_default(true)
     .set_description("Should RGW verify SSL when connecing to a remote HTTP server")