From: Brad Hubbard <bhubbard@redhat.com>
Date: Wed, 17 Apr 2019 06:13:27 +0000 (+1000)
Subject: osd/OSDMap: Replace get_out_osds with get_out_existing_osds
X-Git-Tag: v13.2.7~251^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=a2353be8f04b6ec754ca368b3358d4f2735034ce;p=ceph.git

osd/OSDMap: Replace get_out_osds with get_out_existing_osds

Fixes: http://tracker.ceph.com/issues/39154

Signed-off-by: Brad Hubbard <bhubbard@redhat.com>
(cherry picked from commit adfb6a595363e2c085c6eac3001fcaa2dce007aa)

Conflicts:
	src/osd/OSDMap.h : Resovled for get_out_existing_osds
---

diff --git a/src/mon/OSDMonitor.cc b/src/mon/OSDMonitor.cc
index da9c7d99246..9cee973927d 100644
--- a/src/mon/OSDMonitor.cc
+++ b/src/mon/OSDMonitor.cc
@@ -9762,7 +9762,7 @@ bool OSDMonitor::prepare_command_impl(MonOpRequestRef op,
           (idvec[0] == "any" || idvec[0] == "all" || idvec[0] == "*")) {
         if (prefix == "osd in") {
           // touch out osds only
-          osdmap.get_out_osds(osds);
+          osdmap.get_out_existing_osds(osds);
         } else {
           osdmap.get_all_osds(osds);
         }
diff --git a/src/osd/OSDMap.cc b/src/osd/OSDMap.cc
index 66172314c19..45500af0257 100644
--- a/src/osd/OSDMap.cc
+++ b/src/osd/OSDMap.cc
@@ -1274,10 +1274,10 @@ void OSDMap::get_up_osds(set<int32_t>& ls) const
   }
 }
 
-void OSDMap::get_out_osds(set<int32_t>& ls) const
+void OSDMap::get_out_existing_osds(set<int32_t>& ls) const
 {
   for (int i = 0; i < max_osd; i++) {
-    if (is_out(i))
+    if (exists(i) && get_weight(i) == CEPH_OSD_OUT)
       ls.insert(i);
   }
 }
diff --git a/src/osd/OSDMap.h b/src/osd/OSDMap.h
index e9706f15213..7c6cf41b080 100644
--- a/src/osd/OSDMap.h
+++ b/src/osd/OSDMap.h
@@ -699,7 +699,7 @@ public:
 
   void get_all_osds(set<int32_t>& ls) const;
   void get_up_osds(set<int32_t>& ls) const;
-  void get_out_osds(set<int32_t>& ls) const;
+  void get_out_existing_osds(set<int32_t>& ls) const;
   unsigned get_num_pg_temp() const {
     return pg_temp->size();
   }