From: Laura Flores <lflores@ibm.com>
Date: Thu, 28 Sep 2023 17:52:11 +0000 (+0000)
Subject: osd: fix logic in check_pg_upmaps
X-Git-Tag: v19.0.0~214^2
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=ae11bbe6b2805740d3621cc47c68a5f0da493df0;p=ceph-ci.git

osd: fix logic in check_pg_upmaps

The logic was changed in check_pg_upmaps
in a Reef refactor, which results in recommendations
made by the upmap balancer even when it says there are
no optimizations.

Fixes: https://tracker.ceph.com/issues/63029
Signed-off-by: Laura Flores <lflores@ibm.com>
---

diff --git a/src/osd/OSDMap.cc b/src/osd/OSDMap.cc
index 4a2d400b670..ce199e8ee8a 100644
--- a/src/osd/OSDMap.cc
+++ b/src/osd/OSDMap.cc
@@ -2157,8 +2157,8 @@ bool OSDMap::check_pg_upmaps(
                        << j->first << " " << j->second
                        << dendl;
         to_cancel->push_back(pg);
-      } else {
-        //Josh--check partial no-op here.
+      } else if (newmap != j->second) {
+        // check partial no-op here.
         ldout(cct, 10) << __func__ << " simplifying partially no-op pg_upmap_items "
                        << j->first << " " << j->second
                        << " -> " << newmap