Project

General

Profile

Actions

Feature #58216

closed

cephfs: Add quota.max_files limit check in MDS side

Added by Jinmyeong Lee over 1 year ago. Updated about 1 year ago.

Status:
Rejected
Priority:
Normal
Assignee:
Category:
-
Target version:
-
% Done:

0%

Source:
Tags:
Backport:
Reviewed:
Affected Versions:
Component(FS):
Labels (FS):
Pull request ID:

Description

We are planning to add a quota.max_files limit for each subvolume. (Some malicious user creates tons of file, so this issue triggered a performance degradation in our service.)

While checking the quota feature, I confirmed the cp (or mv) command from the local filesystem to cephfs can copy far more files than the quota.max_files value to the subvolume.

I think a logic to check the dir.rstat is essential when creating a new inode.

$ getfattr -n ceph.quota.max_files /_vsfs_managed/a14c8a8d-4c9a-4da5-b22d-222c8eea2d04 --absolute-name
# file: /_vsfs_managed/a14c8a8d-4c9a-4da5-b22d-222c8eea2d04
ceph.quota.max_files="8" 

[quota_mv]$ sudo cp -R ~/dir_local_1/ ./

[quota_mv]$ tree | wc -l
2440
Actions #2

Updated by Patrick Donnelly over 1 year ago

  • Status changed from New to In Progress
  • Pull request ID set to 49326
Actions #3

Updated by Rishabh Dave about 1 year ago

  • Assignee set to Jinmyeong Lee
Actions #4

Updated by Venky Shankar about 1 year ago

  • Status changed from In Progress to Rejected
Actions

Also available in: Atom PDF