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

mimic: mds: evict cap revoke non-responding clients #24661

Merged
merged 5 commits into from Oct 30, 2018

Conversation

Projects
None yet
5 participants
@vshankar
Copy link
Contributor

vshankar commented Oct 18, 2018

vshankar added some commits Aug 3, 2018

mds: pass timeout argument for fetching late clients
This would be required when fetching clients that have not
responded to cap revoke by MDS for a configured timeout
value.

Additionally, make member functions private which are called
from the Locker class itself.

Signed-off-by: Venky Shankar <vshankar@redhat.com>
(cherry picked from commit 005cf6c)
mds: evict clients that do not respond to cap revoke by MDS
By default, preserve old behaviour. When configured with a non
default value, evict clients that have not responded to cap
revoke by MDS for the configured amount of seconds.

Signed-off-by: Venky Shankar <vshankar@redhat.com>
(cherry picked from commit 4cf7815)

 Conflicts:
	src/common/options.cc
	src/mds/MDSDaemon.cc
	src/mds/MDSRank.h
test: validate client eviction for cap revoke non-responders
Signed-off-by: Venky Shankar <vshankar@redhat.com>
(cherry picked from commit c0b1dac)
mds: add counter for tracking cap non-responding clients
Signed-off-by: Venky Shankar <vshankar@redhat.com>
(cherry picked from commit 8f2de92)

 Conflicts:
	src/mds/Server.cc
doc: document cap revoke non-responders client eviction
Fixes: https://tracker.ceph.com/issues/35975
Signed-off-by: Venky Shankar <vshankar@redhat.com>
(cherry picked from commit 28a52d6)

@vshankar vshankar added the cephfs label Oct 18, 2018

@vshankar vshankar requested review from batrick and smithfarm Oct 18, 2018

@smithfarm smithfarm added this to the mimic milestone Oct 18, 2018

@ukernel ukernel self-requested a review Oct 23, 2018

@yuriw

This comment has been minimized.

Copy link
Contributor

yuriw commented Oct 23, 2018

@yuriw yuriw merged commit 3727626 into ceph:mimic Oct 30, 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment