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/OSDMonitor: check last_osd_report only when the whole cluster is lu… #14294

Merged
merged 1 commit into from Apr 5, 2017

Conversation

Projects
None yet
2 participants
@tchaikov
Contributor

tchaikov commented Apr 3, 2017

…minous

if a peon is not luminous, it won't forward the beacon message. if any
OSD is not luminous, it won't report to mon with beacon messages. so, we
can trust the OSDMonitor::last_osd_report only if the whole cluster is
luminous.

Signed-off-by: Kefu Chai kchai@redhat.com

mon/OSDMonitor: only check last_osd_report if the whole cluster is lu…
…minous

if a peon is not luminous, it won't forward the beacon message. if any
OSD is not luminous, it won't report to mon with beacon messages. so, we
can trust the OSDMonitor::last_osd_report only if the whole cluster is
luminous.

Signed-off-by: Kefu Chai <kchai@redhat.com>

@tchaikov tchaikov changed the title from mon/OSDMonitor: only check last_osd_report if the whole cluster is lu… to mon/OSDMonitor: check last_osd_report only when the whole cluster is lu… Apr 3, 2017

@tchaikov

This comment has been minimized.

@tchaikov tchaikov requested review from liewegas and gregsfortytwo Apr 3, 2017

@liewegas liewegas added the needs-qa label Apr 3, 2017

@liewegas liewegas merged commit d452512 into ceph:master Apr 5, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details

@tchaikov tchaikov deleted the tchaikov:wip-check-last-osd-report-if-luminous branch Apr 5, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment