Skip to content
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.

Sign up for GitHub

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

Jump to bottom

nautilus: mds: change how mds revoke stale caps #28583

Merged
merged 5 commits into from Aug 5, 2019
Merged

nautilus: mds: change how mds revoke stale caps #28583

merged 5 commits into from Aug 5, 2019

Commits on Jul 11, 2019

  1. mds: don't mark unresponsive sessions holding no caps stale 

    When an unresponsive MDS session holds no caps, do not mark it stale
even after session_timeout; at session_autoclose, evict it directly.

Fixes: http://tracker.ceph.com/issues/17854
Signed-off-by: Rishabh Dave <ridave@redhat.com>
(cherry picked from commit 98af31d)
    @rishabh-d-dave @ukernel
    rishabh-d-dave authored and ukernel committed Jul 11, 2019
    Copy the full SHA
    a9dfbad View commit details
    Browse the repository at this point in the history

  2. mds: change how mds revoke stale caps 

    - Only revokes conflicting caps from stale client.
- If stale client holds conflicting CEPH_CAP_ANY_WR,
  blacklist and kill it.

Fixes: https://tracker.ceph.com/issues/38326
Signed-off-by: "Yan, Zheng" <zyan@redhat.com>
(cherry picked from commit dcae1ea)

 Conflicts:
	src/mds/Locker.cc
	src/mds/Server.cc
    @ukernel
    ukernel committed Jul 11, 2019
    Copy the full SHA
    081932f View commit details
    Browse the repository at this point in the history

  3. qa/cephfs: update tests for stale session handling 

    Signed-off-by: "Yan, Zheng" <zyan@redhat.com>
(cherry picked from commit 1c8be58)
    @ukernel
    ukernel committed Jul 11, 2019
    Copy the full SHA
    2e15060 View commit details
    Browse the repository at this point in the history

  4. mds: remove the code that skip evicting the only client 

    There is already logic that defer marking unresponsive client stale.
No reason to defer evicting the only stale client.

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

 Conflicts:
	qa/tasks/cephfs/test_misc.py
    @ukernel
    ukernel committed Jul 11, 2019
    Copy the full SHA
    f2e19bb View commit details
    Browse the repository at this point in the history

  5. mds: check last laggy before marking unresponsive client stale 

    Current mds may evict unresponsive client without going through session
stale. So we need to adjust the last laggy check.

Signed-off-by: "Yan, Zheng" <zyan@redhat.com>
(cherry picked from commit e5cc6f5)
    @ukernel
    ukernel committed Jul 11, 2019
    Copy the full SHA
    44dd1f2 View commit details
    Browse the repository at this point in the history