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/MgrStatMonitor: avoid dup health warnings during luminous upgrade #15986

Merged
merged 1 commit into from Jul 1, 2017

Conversation

Projects
None yet
3 participants
@liewegas
Member

liewegas commented Jun 28, 2017

PGMonitor is still providing warnings; do not expose the ones we are
getting from ceph-mgr (which also come from PGMap::get_health()) too or
else we'll get dups!

Fixes: http://tracker.ceph.com/issues/20435
Signed-off-by: Sage Weil sage@redhat.com

mon/MgrStatMonitor: avoid dup health warnings during luminous upgrade
PGMonitor is still providing warnings; do not expose the ones we are
getting from ceph-mgr (which also come from PGMap::get_health()) too or
else we'll get dups!

Fixes: http://tracker.ceph.com/issues/20435
Signed-off-by: Sage Weil <sage@redhat.com>
@gregsfortytwo

Reviewed-by: Greg Farnum gfarnum@redhat.com

@liewegas liewegas added the needs-qa label Jun 29, 2017

@liewegas liewegas merged commit 9085183 into ceph:master Jul 1, 2017

4 checks passed

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

@liewegas liewegas deleted the liewegas:wip-20435 branch Jul 1, 2017

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