From 0efb04995208c0ce90f2e2ce830fe1c2e7bd205f Mon Sep 17 00:00:00 2001 From: Kefu Chai Date: Fri, 19 Mar 2021 20:20:32 +0800 Subject: [PATCH] osd/PeeringState: remove unused variable recovery_ec_pool_below_min_size was used to verify if the osd in clsuter are octopus and up, but since we are now quincy and up, there is no need to verify this. so drop it for better readability and for silencing the -Wunused-variable warning in Release build. Signed-off-by: Kefu Chai --- src/osd/PeeringState.cc | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/osd/PeeringState.cc b/src/osd/PeeringState.cc index b42400aa942f8..570ebb8b86ec2 100644 --- a/src/osd/PeeringState.cc +++ b/src/osd/PeeringState.cc @@ -2116,9 +2116,6 @@ bool PeeringState::recoverable(const vector &want) const } if (num_want_acting < pool.info.min_size) { - const bool recovery_ec_pool_below_min_size = - HAVE_FEATURE(get_osdmap()->get_up_osd_features(), SERVER_OCTOPUS); - assert(recovery_ec_pool_below_min_size); if (!cct->_conf.get_val("osd_allow_recovery_below_min_size")) { psdout(10) << __func__ << " failed, recovery below min size not enabled" << dendl; return false; -- 2.39.5