New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
nautilus: osd/OSDMap: Replace get_out_osds with get_out_existing_osds #28072
Conversation
Fixes: http://tracker.ceph.com/issues/39154 Signed-off-by: Brad Hubbard <bhubbard@redhat.com> (cherry picked from commit adfb6a5) Conflicts: src/osd/OSDMap.h : Resolved for get_out_existing_osds
retest this please. |
@@ -714,6 +714,7 @@ class OSDMap { | |||
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(std::set<int32_t>& ls) const; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure the resolution is correct here, we should not be using std::
to be consistent.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But it's a header so all references to std:: objects should be fully qualified. There is no "using namespace std" in the file so the lines above it are incorrect and only work by good fortune :) If anything, all the other lines need to be fully qualified but I don't believe that is in the scope of the backport and would tend to obfuscate git history anyway so probably not worth it at this time.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As long as Neha is happy I'm happy.
jenkins retest this please |
jenkins test docs please |
http://tracker.ceph.com/issues/39421