[LU-15839] sanity-quota test_4a: failed create before timer expired Created: 11/May/22 Updated: 27/Jun/22 Resolved: 27/Jun/22 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Lustre 2.16.0 |
| Type: | Bug | Priority: | Minor |
| Reporter: | Maloo | Assignee: | Etienne Aujames |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Severity: | 3 |
| Rank (Obsolete): | 9223372036854775807 |
| Description |
|
This issue was created by maloo for eaujames <eaujames@ddn.com> This issue relates to the following test suite run: https://testing.whamcloud.com/test_sets/5f3ccc4f-90c6-4ec4-89ff-c766a6cf0413 test_4a failed with the following error: failed create before timer expired, but expect success. 1652231106, 1652231123 The test was running with ZFS. VVVVVVV DO NOT REMOVE LINES BELOW, Added by Maloo for auto-association VVVVVVV |
| Comments |
| Comment by Etienne Aujames [ 11/May/22 ] |
|
For ZFS, it seems that the grace period is modified to 20s after setting the value to 12s on the filesystem: test_4a() {
....
local GRACE=12
....
$LFS setquota -t -u --block-grace $MAX_DQ_TIME --inode-grace \
$GRACE $DIR || error "set user grace time failed" <------ grace is set here on fs
$LFS setquota -u $TSTUSR -b 0 -B 0 -i $LIMIT -I 0 $DIR ||
error "set user quota failed"
[ "$mds1_FSTYPE" = zfs ] && GRACE=20 <------ grace is modified here for ZFS
test_file_soft $TESTFILE $LIMIT $GRACE "u"
So the test could fail between 12-20s for ZFS (1652231123 - 1652231106 = 17s): test_file_soft() {
....
sync_all_data || true
local cur_time=$(date +%s)
[ $(($cur_time - $trigger_time)) -ge $grace ] && <---- grace=20s
error "Passed grace time $grace, $trigger_time, $cur_time"
echo "Create file before timer goes off"
$RUNAS touch ${TESTFILE}_before ||
quota_error a $TSTUSR "failed create before timer expired," \ <---- fs grace=12s
"but expect success. $trigger_time, $cur_time"
....
|
| Comment by Gerrit Updater [ 11/May/22 ] |
|
"Etienne AUJAMES <eaujames@ddn.com>" uploaded a new patch: https://review.whamcloud.com/47289 |
| Comment by Gerrit Updater [ 27/Jun/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/47289/ |
| Comment by Peter Jones [ 27/Jun/22 ] |
|
Landed for 2.16 |