From: Alex Zhuravlev Date: Tue, 15 Apr 2025 08:58:08 +0000 (+0300) Subject: LU-18293 tests: Revert "LU-18293 test: use direct IO for" X-Git-Tag: 2.16.55~110 X-Git-Url: https://git.whamcloud.com/gitweb?a=commitdiff_plain;h=refs%2Fchanges%2F90%2F58790%2F2;p=fs%2Flustre-release.git LU-18293 tests: Revert "LU-18293 test: use direct IO for" This reverts commit 2f96622a522924c7b9e26fd2dfdf34eb6227b7a6 as it causes many failures in sanity-quota/71a Signed-off-by: Alex Zhuravlev Change-Id: I94d5ca67db64c0eead079ae415455f9c5951f9ed Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/58790 Reviewed-by: Sebastien Buisson Reviewed-by: James Simmons Reviewed-by: Andreas Dilger Tested-by: jenkins Tested-by: Maloo --- diff --git a/lustre/tests/sanity-quota.sh b/lustre/tests/sanity-quota.sh index ffbaeea..b580ccd 100755 --- a/lustre/tests/sanity-quota.sh +++ b/lustre/tests/sanity-quota.sh @@ -566,13 +566,12 @@ test_1_check_write() { local short_qtype=${qtype:0:1} log "Write..." - $RUNAS $DD of=$testfile count=$((limit/2)) oflag=direct || + $RUNAS $DD of=$testfile count=$((limit/2)) || quota_error $short_qtype $TSTUSR \ "$qtype write failure, but expect success" log "Write out of block quota ..." # this time maybe cache write, ignore it's failure - $RUNAS $DD of=$testfile count=$((limit/2)) seek=$((limit/2)) \ - oflag=direct || true + $RUNAS $DD of=$testfile count=$((limit/2)) seek=$((limit/2)) || true # flush cache, ensure noquota flag is set on client cancel_lru_locks osc sync; sync_all_data || true @@ -580,7 +579,7 @@ test_1_check_write() { # guarantee that slave received new edquot through glimpse. # so wait a little to be sure slave got it. sleep 5 - $RUNAS $DD of=$testfile count=1 seek=$limit oflag=direct && + $RUNAS $DD of=$testfile count=1 seek=$limit && quota_error $short_qtype $TSTUSR \ "user write success, but expect EDQUOT" return 0