From: Igor Golikov <igolikov@redhat.com>
Date: Tue, 10 Feb 2026 07:58:10 +0000 (+0000)
Subject: mds: remove redundant validation of parameter
X-Git-Tag: testing/wip-vshankar-testing-20260222.101816~3^2~3
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=2da10e7e9a8a00a5e732f0bdb5cfd4f6209e7128;p=ceph-ci.git

mds: remove redundant validation of parameter

subv_window_sec will be always set to default, if not set explicitly

Signed-off-by: Igor Golikov <igolikov@redhat.com>
---

diff --git a/src/mds/MDCache.cc b/src/mds/MDCache.cc
index 1ca5cd33b86..3b267cb967c 100644
--- a/src/mds/MDCache.cc
+++ b/src/mds/MDCache.cc
@@ -2203,7 +2203,6 @@ update:
     msg->quota = pi->quota;
     mds->send_message_client_counted(msg, cap->get_session());
   }
-
   for (const auto &it : in->get_replicas()) {
     auto msg = make_message<MGatherCaps>();
     msg->ino = in->ino();
diff --git a/src/mds/MetricAggregator.cc b/src/mds/MetricAggregator.cc
index 6d9e04e8247..183b9e9c689 100644
--- a/src/mds/MetricAggregator.cc
+++ b/src/mds/MetricAggregator.cc
@@ -111,11 +111,6 @@ int MetricAggregator::init() {
     });
 
   subv_window_sec = g_conf().get_val<std::chrono::seconds>("subv_metrics_window_interval").count();
-  if (!subv_window_sec) {
-    dout(0) << "subv_metrics_window_interval is not set, setting to 300 seconds" << dendl;
-    subv_window_sec = 300;
-  }
- 
   return 0;
 }
 
diff --git a/src/mds/MetricsHandler.cc b/src/mds/MetricsHandler.cc
index 63328e40ed7..61f5b88771a 100644
--- a/src/mds/MetricsHandler.cc
+++ b/src/mds/MetricsHandler.cc
@@ -86,10 +86,6 @@ void MetricsHandler::init() {
   }
 
   subv_window_sec = g_conf().get_val<std::chrono::seconds>("subv_metrics_window_interval").count();
-  if (!subv_window_sec) {
-    dout(0) << "subv_metrics_window_interval is not set, setting to 300 seconds" << dendl;
-    subv_window_sec = 300;
-  }
 
   updater = std::thread([this]() {
       ceph_pthread_setname("mds-metrics");