Project

General

Profile

Feature #58216

cephfs: Add quota.max_files limit check in MDS side

Added by Jinmyeong Lee about 1 year ago. Updated 11 months 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

History

#2 Updated by Patrick Donnelly about 1 year ago

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

#3 Updated by Rishabh Dave 12 months ago

  • Assignee set to Jinmyeong Lee

#4 Updated by Venky Shankar 11 months ago

  • Status changed from In Progress to Rejected

Also available in: Atom PDF