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
hammer: mon: map_cache can become inaccurate if osd does not receive the osdmaps #5773
Conversation
@dachary turns out we should backport "less" than you did in #5756.
|
needs rebase |
4808e21
to
8dd88b0
Compare
* remove osd_epoch<osd, epoch> from OSDMonitor * add osd_epoch to MonSession to track the latest osdmap epoch OSDMonitor sends to a mon client * do not remove osd_epoch entries if an OSD is down, or max_osd > osd_id Fixes: ceph#10930 Signed-off-by: Kefu Chai <kchai@redhat.com> (cherry picked from commit c05753e)
previously, we only track the osd_epoch for OSD peers. but other MonClients who receives osdmaps can also benefit from osd_epoch. Signed-off-by: Kefu Chai <kchai@redhat.com> (cherry picked from commit de43a02) Conflicts: src/mon/OSDMonitor.cc: do not assume that the MonSession is always available when OSDMonitor::send_incremental() is called. as when the connection to the mon client is dropped, the Monitor is aware of this, but not the OSDMonitor.
8dd88b0
to
171fee1
Compare
removed the assertion for the MonSession, will rerun some of the failed test once the build is ready. |
@dachary the test passed at http://pulpito.ceph.com/kchai-2015-11-03_00:33:48-rados:thrash-wip-5773-hammer---basic-multi/ could you take a look at this pr again? thanks! |
@tchaikov ok, thanks ! |
… does not receive the osdmaps Reviewed-by: Loic Dachary <ldachary@redhat.com>
… does not receive the osdmaps Reviewed-by: Loic Dachary <ldachary@redhat.com>
… does not receive the osdmaps Reviewed-by:
… does not receive the osdmaps Reviewed-by: Loic Dachary <ldachary@redhat.com>
… does not receive the osdmaps Reviewed-by: Loic Dachary <ldachary@redhat.com>
… does not receive the osdmaps Reviewed-by: Loic Dachary <ldachary@redhat.com>
… does not receive the osdmaps Reviewed-by: Loic Dachary <ldachary@redhat.com>
It passed a run of the hammer rados suite ( see http://tracker.ceph.com/issues/13356#note-28 for details ). |
mon: map_cache can become inaccurate if osd does not receive the osdmaps Reviewed-by: Loic Dachary <ldachary@redhat.com>
http://tracker.ceph.com/issues/12835