From a2353be8f04b6ec754ca368b3358d4f2735034ce Mon Sep 17 00:00:00 2001 From: Brad Hubbard Date: Wed, 17 Apr 2019 16:13:27 +1000 Subject: [PATCH] osd/OSDMap: Replace get_out_osds with get_out_existing_osds Fixes: http://tracker.ceph.com/issues/39154 Signed-off-by: Brad Hubbard (cherry picked from commit adfb6a595363e2c085c6eac3001fcaa2dce007aa) Conflicts: src/osd/OSDMap.h : Resovled for get_out_existing_osds --- src/mon/OSDMonitor.cc | 2 +- src/osd/OSDMap.cc | 4 ++-- src/osd/OSDMap.h | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/mon/OSDMonitor.cc b/src/mon/OSDMonitor.cc index da9c7d992464..9cee973927d1 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 66172314c193..45500af0257c 100644 --- a/src/osd/OSDMap.cc +++ b/src/osd/OSDMap.cc @@ -1274,10 +1274,10 @@ void OSDMap::get_up_osds(set& ls) const } } -void OSDMap::get_out_osds(set& ls) const +void OSDMap::get_out_existing_osds(set& 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 e9706f15213c..7c6cf41b080d 100644 --- a/src/osd/OSDMap.h +++ b/src/osd/OSDMap.h @@ -699,7 +699,7 @@ public: void get_all_osds(set& ls) const; void get_up_osds(set& ls) const; - void get_out_osds(set& ls) const; + void get_out_existing_osds(set& ls) const; unsigned get_num_pg_temp() const { return pg_temp->size(); } -- 2.47.3