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
mon: post-luminous cleanup (part 3 of ?) #17607
Conversation
3bb0e8d
to
3c2ae4b
Compare
will rebase |
Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
3c2ae4b
to
b81715b
Compare
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.
This is mostly okay, just a few things I think got in by accident.
@@ -1776,7 +1776,8 @@ void Monitor::handle_probe_reply(MonOpRequestRef op) | |||
<< " upgrading" << dendl; | |||
exit(0); | |||
} | |||
if (!osdmon()->osdmap.test_flag(CEPH_OSDMAP_PURGED_SNAPDIRS)) { | |||
if (!osdmon()->osdmap.test_flag(CEPH_OSDMAP_PURGED_SNAPDIRS) || | |||
!osdmon()->osdmap.test_flag(CEPH_OSDMAP_RECOVERY_DELETES)) { |
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.
Is RECOVERY_DELETES actually set as part of a scrub? I don't know how that works.
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.
Once the (luminous) mon sees that all legacy_snapdirs have been removed it sets the flag. The mimic mons refuse to start up if that flag isn't set yet (i.e., if they didn't stay on luminous long enough to do the conversion).
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.
src/mon/OSDMonitor.cc
Outdated
@@ -7481,16 +7474,6 @@ bool OSDMonitor::prepare_command_impl(MonOpRequestRef op, | |||
goto reply; | |||
} | |||
if (prefix == "osd crush weight-set create") { | |||
if (osdmap.require_min_compat_client > 0 && | |||
osdmap.require_min_compat_client < CEPH_RELEASE_LUMINOUS) { |
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.
Surely we don't mean to require Luminous clients to talk to Mimic clusters?
src/mon/OSDMonitor.cc
Outdated
err = -EPERM; | ||
goto reply; | ||
} | ||
if (osdmap.require_min_compat_client < CEPH_RELEASE_LUMINOUS) { |
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.
Again, client.
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.
good catch, fixed!
Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
…ions Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
We may not end up doing this but it keeps a template for how the guard worked before in the code for future reference. Signed-off-by: Sage Weil <sage@redhat.com>
b81715b
to
8ade1af
Compare
@gregsfortytwo ok to merge? |
jenkins test docs |
Yep, looks good @liewegas! |
thanks! |
No description provided.