]> git.apps.os.sepia.ceph.com Git - ceph.git/commit
client: correct quota check in Client::_rename() 50127/head
authorRishabh Dave <ridave@redhat.com>
Wed, 15 Feb 2023 13:19:10 +0000 (18:49 +0530)
committerRishabh Dave <ridave@redhat.com>
Wed, 22 Feb 2023 16:30:47 +0000 (22:00 +0530)
commita8d9a256dc1c7f4186f293eb25c4edc54f69eda3
treeaab3ac7ea0dd39b767a684fabbdb108c35373bbd
parenta29a620b873995179c547f98fcabdd93ba8bae46
client: correct quota check in Client::_rename()

Client::_rename() only checks for max files quota. As a result, max
bytes quota is not enforced. Thus the following code (copied from
qa/workunits/fs/quota/quota.sh), runs successfully instead of exiting
with error "Disk quota exceeded" -

mkdir files limit
setfattr limit -n ceph.quota.max_bytes -v 1000
truncate filles/file1 -s 1M
mv files limit

This patch makes Client::_rename() enforce both kinds of quota which
leads command "mv files limit" from above script to fail with "Disk
quota exceeded" error. This is the expected behaviour in this case.

Introduced-by: dde1a19d4105f1db41477752a71d8fe40c45cd7f
Fixes: https://tracker.ceph.com/issues/58220
Signed-off-by: Rishabh Dave <ridave@redhat.com>
src/client/Client.cc