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
jewel: mon: osdmonitor: decouple adjust_heartbeat_grace and min_down_reporters #10757
Conversation
if cancel mon_osd_adjust_heartbeat_grace, we must set mon_osd_min_down_reporters to zero. otherwise the next checking for reporters_by_subtree.size() >= g_conf->mon_osd_min_down_reporters will allways be fail.the two options should take effect respectively... Signed-off-by: Zengran Zhang <zhangzengran@h3c.com> (cherry picked from commit 7ac5ca5)
…rtbeat_grace and min_down_reporters Reviewed-by: Loic Dachary <ldachary@redhat.com>
…rtbeat_grace and min_down_reporters Reviewed-by: Loic Dachary <ldachary@redhat.com>
…rtbeat_grace and min_down_reporters Reviewed-by: Loic Dachary <ldachary@redhat.com>
…rtbeat_grace and min_down_reporters Reviewed-by: Loic Dachary <ldachary@redhat.com>
…rtbeat_grace and min_down_reporters Reviewed-by: Loic Dachary <ldachary@redhat.com>
…rtbeat_grace and min_down_reporters Reviewed-by: Loic Dachary <ldachary@redhat.com>
…rtbeat_grace and min_down_reporters Reviewed-by: Loic Dachary <ldachary@redhat.com>
…rtbeat_grace and min_down_reporters Reviewed-by: Loic Dachary <ldachary@redhat.com>
…rtbeat_grace and min_down_reporters Reviewed-by: Loic Dachary <ldachary@redhat.com>
…rtbeat_grace and min_down_reporters Reviewed-by: Loic Dachary <ldachary@redhat.com>
@tchaikov does this backport look good to merge ? It passed the rados suite http://tracker.ceph.com/issues/17487#note-4 (except for one known bug that is, I believe, unrelated and one environmental failure on an erasure code job). |
…rtbeat_grace and min_down_reporters Reviewed-by: Loic Dachary <ldachary@redhat.com>
…rtbeat_grace and min_down_reporters Reviewed-by: Loic Dachary <ldachary@redhat.com>
…rtbeat_grace and min_down_reporters Reviewed-by: Loic Dachary <ldachary@redhat.com>
It passed the rados, upgrade/jewel-x and upgrade/hammer-x suites (except for two bugs that are, I believe, unrelated) |
…rtbeat_grace and min_down_reporters Reviewed-by: Loic Dachary <ldachary@redhat.com>
…rtbeat_grace and min_down_reporters Reviewed-by: Loic Dachary <ldachary@redhat.com>
It passed the rados (http://tracker.ceph.com/issues/17487#note-19) suite except for two jobs that are, I believe unrelated. It also passed the upgrade/jewel-x and upgrade/hammer-x (http://tracker.ceph.com/issues/17487#note-22) suites. |
lgtm. |
http://tracker.ceph.com/issues/17056