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
mds: add command that config individual client session #29104
Conversation
7eed172
to
938f7b9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please also add a trivial test to test_client_recovery.py
.
For now, the command only can change session's timeout config Fixes: http://tracker.ceph.com/issues/40811 Signed-off-by: "Yan, Zheng" <zyan@redhat.com>
5dc0811
to
792477b
Compare
self.fs.mds_asok(['session', 'config', '%s' % mount_a_gid, 'timeout', '%s' % (session_timeout * 2)]) | ||
|
||
self.mount_a.kill(); | ||
self.mount_a.kill_cleanup() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
kill_cleanup
now waits for the hard reboot to complete: #28825
I guess the right thing to do here is to add a sleep after kill()
, assert 2 sessions, then do kill_cleanup()
.
See this failure:
/ceph/teuthology-archive/pdonnell-2019-07-26_06:38:30-kcephfs-wip-pdonnell-testing-20190726.021409-distro-basic-smithi/4152101/teuthology.log
Signed-off-by: "Yan, Zheng" <zyan@redhat.com>
* refs/pull/29104/head: qa/cephfs: test case for timeout config of individual session mds: add command that config individual client session Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
For now, the command only can change session's timeout config
Fixes: http://tracker.ceph.com/issues/40811
Signed-off-by: "Yan, Zheng" zyan@redhat.com