]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
mon: osdmon: simplify maybe_rm_from_pending_blocklists
authorGreg Farnum <gfarnum@redhat.com>
Thu, 28 Oct 2021 23:04:23 +0000 (23:04 +0000)
committerGreg Farnum <gfarnum@redhat.com>
Tue, 31 May 2022 23:18:16 +0000 (23:18 +0000)
Signed-off-by: Greg Farnum <gfarnum@redhat.com>
(cherry picked from commit 871427881a60f7a203d08373a1ae1e6db9e2976b)

src/mon/OSDMonitor.cc

index db5ea6929de01de7756c649813e6269507aef644..17b087932515bbae79475fff219b68ce8c08c8cc 100644 (file)
@@ -12752,16 +12752,14 @@ bool OSDMonitor::prepare_command_impl(MonOpRequestRef op,
        auto maybe_rm_from_pending_blocklists = [](const auto& addr,
                                                   auto& blocklist,
                                                   auto& ob, auto& pb) {
-         if (blocklist.count(addr) ||
-             pb.count(addr)) {
-           if (blocklist.count(addr))
-             ob.push_back(addr);
-           else
-             pb.erase(addr);
+         if (blocklist.count(addr)) {
+           ob.push_back(addr);
+           return true;
+         } else if (pb.count(addr)) {
+           pb.erase(addr);
            return true;
-         } else {
-           return false;
          }
+         return false;
        };
        if (maybe_rm_from_pending_blocklists(addr, osdmap.blocklist,
                                             pending_inc.old_blocklist,