[LU-12969] make sure we reset quota limits for all test ID/users

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.14.0
• Severity: 3
• Assignee: Wang Shilong (Inactive)
• Reporter: Wang Shilong (Inactive)

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

[Peter Jones]

Landed for 2.14

[Gerrit Updater]

Oleg Drokin ([e-mail]) 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

[Peter Jones]

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