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

luminous: cephfs: MDSMonitor: consider raising priority of MMDSBeacons from MDS so they are processed before other client messages #23554

Merged
merged 1 commit into from Sep 17, 2018

Conversation

Projects
None yet
3 participants
@batrick
Member

batrick commented Aug 13, 2018

mds: mark beacons as high priority
The mons already mark beacon replies as high priority (via default mon message
priority). We should expect that the mons handle our beacons at the same
priority so the MDS doesn't wrongly get marked laggy.

Fixes: http://tracker.ceph.com/issues/26899

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
(cherry picked from commit ec3fa7b)

@batrick batrick added the cephfs label Aug 13, 2018

@smithfarm smithfarm added this to the luminous milestone Aug 28, 2018

@yuriw

This comment has been minimized.

Contributor

yuriw commented Sep 13, 2018

@batrick

This comment has been minimized.

Member

batrick commented Sep 17, 2018

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>

@yuriw yuriw merged commit 7180037 into ceph:luminous Sep 17, 2018

4 checks passed

Docs: build check OK - docs built
Details
Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details

@batrick batrick deleted the batrick:i26906 branch Sep 26, 2018

@smithfarm smithfarm changed the title from luminous: MDSMonitor: consider raising priority of MMDSBeacons from MDS so they are processed before other client messages to luminous: cephfs: MDSMonitor: consider raising priority of MMDSBeacons from MDS so they are processed before other client messages Oct 26, 2018

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