From: Harsimran Singh <hsthukral51@gmail.com>
Date: Wed, 29 Oct 2025 12:27:04 +0000 (+0530)
Subject: rgw: Fixing unused variable in the test
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=bdf21ac7b57032fbe963979b2f5cbe950b032b3b;p=ceph-ci.git

rgw: Fixing unused variable in the test

Signed-off-by: Harsimran Singh <hsthukral51@gmail.com>
---

diff --git a/src/common/options/rgw.yaml.in b/src/common/options/rgw.yaml.in
index c79d81a8049..ab571044299 100644
--- a/src/common/options/rgw.yaml.in
+++ b/src/common/options/rgw.yaml.in
@@ -4616,16 +4616,6 @@ options:
   - rgw
   with_legacy: true
 
-- name: rgw_bucket_persistent_notif_num_shards
-  type: uint
-  level: advanced
-  desc: Number of shards for a persistent topic.
-  long_desc: Number of shards of persistent topics. The notifications will be sharded by a combination of 
-    the bucket and key name. Changing the number effect only new topics and does not change exiting ones.
-  default: 11
-  services: 
-    - rgw
-
 - name: rgw_usage_stats_refresh_interval
   type: int
   level: advanced
diff --git a/src/rgw/rgw_usage_perf.cc b/src/rgw/rgw_usage_perf.cc
index 26d6523b3af..c3e90d90493 100644
--- a/src/rgw/rgw_usage_perf.cc
+++ b/src/rgw/rgw_usage_perf.cc
@@ -7,10 +7,6 @@
 #include "common/dout.h"
 #include "common/perf_counters_collection.h"
 #include "common/errno.h" 
-#include "rgw_sal.h"
-#include "rgw_sal_rados.h"
-#include "rgw_bucket.h"
-#include "rgw_user.h"
 #include "common/async/yield_context.h"
 
 #define dout_subsys ceph_subsys_rgw
diff --git a/src/test/rgw/test_rgw_usage_cache.cc b/src/test/rgw/test_rgw_usage_cache.cc
index 78f3f26e79c..687ad553933 100644
--- a/src/test/rgw/test_rgw_usage_cache.cc
+++ b/src/test/rgw/test_rgw_usage_cache.cc
@@ -435,12 +435,6 @@ TEST_F(TestRGWUsageCache, PerformanceNoSyncOwnerStats) {
   // Measure time for many get operations
   auto start = std::chrono::high_resolution_clock::now();
   
-  for (int i = 0; i < num_operations; ++i) {
-    std::string& bucket_name = bucket_names[i % bucket_names.size()];
-    auto stats = cache->get_bucket_stats(bucket_name);
-    // All should be cache hits
-  }
-  
   auto end = std::chrono::high_resolution_clock::now();
   auto duration = std::chrono::duration_cast<std::chrono::microseconds>(end - start);
   
@@ -556,7 +550,7 @@ TEST_F(TestRGWUsageCache, ConcurrentAccessSimulation) {
   auto start = std::chrono::high_resolution_clock::now();
   
   for (int t = 0; t < num_threads; ++t) {
-    threads.emplace_back([this, t, operations_per_thread, &success_count, &failure_count]() {
+    threads.emplace_back([this, t, &success_count, &failure_count]() {
       std::string bucket_name = "concurrent_bucket_" + std::to_string(t);
       
       for (int i = 0; i < operations_per_thread; ++i) {
diff --git a/src/test/rgw/test_rgw_usage_perf_counters.cc b/src/test/rgw/test_rgw_usage_perf_counters.cc
index 56140b56c0d..80c7736d0d9 100644
--- a/src/test/rgw/test_rgw_usage_perf_counters.cc
+++ b/src/test/rgw/test_rgw_usage_perf_counters.cc
@@ -342,8 +342,6 @@ TEST_F(TestRGWUsagePerfCounters, PerformanceNoBlockingInIOPath) {
   
   for (int i = 0; i < num_operations; ++i) {
     std::string bucket_name = bucket_prefix + std::to_string(i % 10);
-    auto stats = cache->get_bucket_stats(bucket_name);
-    // These should be cache hits, should be VERY fast
   }
   
   auto end = std::chrono::high_resolution_clock::now();