[LU-12969] make sure we reset quota limits for all test ID/users Created: 14/Nov/19  Updated: 16/Dec/19  Resolved: 16/Dec/19

Status: Resolved
Project: Lustre
Component/s: None
Affects Version/s: None
Fix Version/s: Lustre 2.14.0

Type: Bug Priority: Minor
Reporter: Wang Shilong (Inactive) Assignee: Wang Shilong (Inactive)
Resolution: Fixed Votes: 0
Labels: None

Severity: 3
Rank (Obsolete): 9223372036854775807

 Description   

It looks current sanity-quota.sh assumed TSTID/TSTID2 mapped with quota_usr/quota_2usr.
However, in a real testing envirment this might be not true, and in Oleg's testing enviroment,
trying to add a new test66 which did not set any limits before test, but hit disk quota failure:

== sanity-quota test 66: nonroot user can not change project state in default ======================== 02:15:34 (1573715734)
Waiting for local destroys to complete
Creating test directory
fail_val=0
fail_loc=0
mdt.lustre-MDT0000.enable_chprojid_gid=0
mdt.lustre-MDT0001.enable_chprojid_gid=0
striped dir -i0 -c2 /mnt/lustre/d66.sanity-quota/foo
Disk quotas for usr 60000 (uid 60000):
     Filesystem  kbytes   quota   limit   grace   files   quota   limit   grace
/mnt/lustre/d66.sanity-quota/foo
                      0       0       0       -       0       0       0       -
Disk quotas for grp 60000 (gid 60000):
     Filesystem  kbytes   quota   limit   grace   files   quota   limit   grace
/mnt/lustre/d66.sanity-quota/foo
                      0    1000    1000       -       0    1000    1000       -
Disk quotas for prj 1000 (pid 1000):
     Filesystem  kbytes   quota   limit   grace   files   quota   limit   grace
/mnt/lustre/d66.sanity-quota/foo
                      0       0       0       -       0       0       0       -
lfs project -sp 1000 /mnt/lustre/d66.sanity-quota/foo
running as uid/gid/euid/egid 60000/60000/60000/60000, groups:
 [touch] [/mnt/lustre/d66.sanity-quota/foo/foo]
touch: cannot touch '/mnt/lustre/d66.sanity-quota/foo/foo': Disk quota exceeded
 sanity-quota test_66: @@@@@@ FAIL: failed to touch foo 
  Trace dump:
  = /home/green/git/lustre-release/lustre/tests/test-framework.sh:6108:error()
  = /home/green/git/lustre-release/lustre/tests/sanity-quota.sh:3492:test_66()
  = /home/green/git/lustre-release/lustre/tests/test-framework.sh:6410:run_one()
  = /home/green/git/lustre-release/lustre/tests/test-framework.sh:6449:run_one_logged()
  = /home/green/git/lustre-release/lustre/tests/test-framework.sh:6295:run_test()
  = /home/green/git/lustre-release/lustre/tests/sanity-quota.sh:3516:main()
Dumping lctl log to /tmp/testlogs//sanity-quota.test_66.*.1573715738.log
oleg15-server: Warning: Permanently added 'oleg15-client.virtnet' (ECDSA) to the list of known hosts.
oleg15-server: rsync: chown "/tmp/testlogs/.sanity-quota.test_66.debug_log.oleg15-server.1573715738.log.PKpyYF" failed: Operation not permitted (1)
oleg15-server: rsync: chown "/tmp/testlogs/.sanity-quota.test_66.dmesg.oleg15-server.1573715738.log.hsd0qy" failed: Operation not permitted (1)
oleg15-server: rsync error: some files/attrs were not transferred (see previous errors) (code 23) at main.c(1178) [sender=3.1.2]
pdsh@oleg15-client: oleg15-server: ssh exited with exit code 23
Resetting fail_loc on all nodes...done.
mdt.lustre-MDT0000.enable_chprojid_gid=0
mdt.lustre-MDT0001.enable_chprojid_gid=0
Delete files...
Wait for unlink objects finished...
Waiting for local destroys to complete



 Comments   
Comment by Peter Jones [ 15/Dec/19 ]

https://review.whamcloud.com/#/c/36558/

Comment by Gerrit Updater [ 16/Dec/19 ]

Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/36756/
Subject: LU-12969 test: reset quota limits for all test ID/users
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 2953ff3dcefe6c6d958a28c3d0e8739fd22a6c1d

Comment by Peter Jones [ 16/Dec/19 ]

Landed for 2.14

Generated at Sat Feb 10 02:57:14 UTC 2024 using Jira 9.4.14#940014-sha1:734e6822bbf0d45eff9af51f82432957f73aa32c.