Project

General

Profile

Actions

Bug #62257

open

Bug #61947: mds: enforce a limit on the size of a session in the sessionmap

mds: blocklist clients that are not advancing `oldest_client_tid`

Added by Venky Shankar 9 months ago. Updated 9 months ago.

Status:
New
Priority:
Normal
Assignee:
Category:
Administration/Usability
Target version:
% Done:

0%

Source:
Tags:
Backport:
reef,quincy,pacific
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(FS):
MDS
Labels (FS):
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

The size of the session map becomes huge, thereby exceeding the max write size for a RADOS operation, thereby resulting in the MDS to go read-only since the RADOS op failed.

For writing tests, it might be helpful to introduce a test config that emulates the bug. FWIW, config `client_inject_fixed_oldest_tid` already exists that mangles oldest_client_tid, so, maybe that can be reused for this.

Actions #1

Updated by Xiubo Li 9 months ago

There is a ceph-user mail thread about this https://www.spinics.net/lists/ceph-users/msg78109.html.

As a workaround we can evict the client manually.

Actions #2

Updated by Venky Shankar 9 months ago

Xiubo Li wrote:

There is a ceph-user mail thread about this https://www.spinics.net/lists/ceph-users/msg78109.html.

As a workaround we can evict the client manually.

Yes, that's a workaround, but until the bug is found having the MDS blocklist the client will ensure that the MDS does not go read-only (which causes service downtime).

Actions #3

Updated by Xiubo Li 9 months ago

Venky Shankar wrote:

Xiubo Li wrote:

There is a ceph-user mail thread about this https://www.spinics.net/lists/ceph-users/msg78109.html.

As a workaround we can evict the client manually.

Yes, that's a workaround, but until the bug is found having the MDS blocklist the client will ensure that the MDS does not go read-only (which causes service downtime).

Agree.

Actions #4

Updated by Venky Shankar 9 months ago

  • Assignee changed from Leonid Usov to Venky Shankar
Actions

Also available in: Atom PDF