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: mds: reset heartbeat map at potential time-consuming places #23507

Merged
merged 2 commits into from Sep 17, 2018

Conversation

Projects
None yet
5 participants
@ukernel
Member

ukernel commented Aug 9, 2018

ukernel added some commits Jul 11, 2018

mds: reset heartbeat map at potential time-consuming places
Signed-off-by: Yan, Zheng <zyan@redhat.com>
Fixes: http://tracker.ceph.com/issues/26858
(cherry picked from commit 5a6a9a3)

 Conflicts:
	src/mds/Migrator.cc
	src/mds/MDSRank.h
mds: consider max age of dispatch queue when finding stale client
session renew messages from clients can be in the dispatch queue,
waiting for getting dispatched.

Signed-off-by: "Yan, Zheng" <zyan@redhat.com>
(cherry picked from commit 2ca5708)

 Conflicts:
	src/mds/Server.cc

@tchaikov tchaikov added this to the luminous milestone Aug 9, 2018

@smithfarm smithfarm added the cephfs label Sep 11, 2018

@smithfarm smithfarm requested a review from batrick Sep 11, 2018

@yuriw

This comment has been minimized.

Contributor

yuriw commented Sep 13, 2018

@yuriw yuriw merged commit 786e77e 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
@smithfarm

This comment has been minimized.

Contributor

smithfarm commented Sep 25, 2018

@ukernel @batrick I updated the PR description to reflect that this PR resolves two backport issues. Please double-check that this is correct.

@batrick

This comment has been minimized.

Member

batrick commented Sep 25, 2018

Looks like they were duplicates. I closed them.

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