[LU-2233] sanityn/40-46 takes ~800-900s Created: 25/Oct/12  Updated: 30/Jan/22  Resolved: 10/May/19

Status: Resolved
Project: Lustre
Component/s: None
Affects Version/s: None
Fix Version/s: Lustre 2.13.0, Lustre 2.12.9

Type: Improvement Priority: Minor
Reporter: Alex Zhuravlev Assignee: Alex Zhuravlev
Resolution: Fixed Votes: 0
Labels: patch

Issue Links:
Related
is related to LU-10754 sanityn test 47b fails with 'create m... Resolved
Rank (Obsolete): 5291

 Description   

tests 40-46 from sanityn takes ~800-900s, which is almost a half of the whole sanityn pass.



 Comments   
Comment by Alex Zhuravlev [ 26/Oct/12 ]

https://maloo.whamcloud.com/test_sets/adb04eb8-1f33-11e2-bae2-52540035b04c – sanityn took 639s. usually it takes 1300-1700s.

Comment by Andreas Dilger [ 16/Sep/17 ]
LU-2233 tests: improve tests sanityn/40-46

sanity/40-46 usually take 800-900s which is almost a half
of the whole sanityn pass. 99.(9)% of time the tests just
wait to ensure specific order the operations execute in.

cfs_fail_intr is introduced to interrupt OBD_FAIL_TIMEOUT().
this way the tests can unblock the threads to resumt the
execution.

the tests itself are modified to use cfs_fail_intr. to be
able to do so both operations (referenced as OP1 and OP2
in the tests) are run in background. once started and then
ensured with pdo_sched() helper that MDS threads got to the
blocking points, we can interrupt OP1 and do usual checks.

on the local setup sanityn take: 1076s and 524s after.

Signed-off-by: Alex Zhuravlev <alexey.zhuravlev@intel.com>

Reviewed-on: https://review.whamcloud.com/4392
Comment by Gerrit Updater [ 10/May/19 ]

Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/4392/
Subject: LU-2233 tests: improve tests sanityn/40-47
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 743b85a32e24cff0c77dff739691043970a0901e

Comment by Peter Jones [ 10/May/19 ]

Landed for 2.13

Comment by Gerrit Updater [ 10/May/19 ]

Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/34847
Subject: Revert "LU-2233 tests: improve tests sanityn/40-47"
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: dca71e339b4c906d61107972c85ff323b7ca438d

Comment by Gerrit Updater [ 12/Feb/21 ]

Etienne AUJAMES (eaujames@ddn.com) uploaded a new patch: https://review.whamcloud.com/41660
Subject: LU-2233 tests: improve tests sanityn/40-47
Project: fs/lustre-release
Branch: b2_12
Current Patch Set: 1
Commit: 1f58da293f94bfb751ef1f6618cc120df25ed801

Comment by Gerrit Updater [ 23/Jul/21 ]

Etienne AUJAMES (eaujames@ddn.com) uploaded a new patch: https://review.whamcloud.com/44391
Subject: LU-2233 tests: improve tests sanityn/40-47
Project: fs/lustre-release
Branch: b2_12
Current Patch Set: 1
Commit: ee441c41d9f076456b68f45f9c8e116c450dc38c

Comment by Gerrit Updater [ 30/Jan/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/44391/
Subject: LU-2233 tests: improve tests sanityn/40-47
Project: fs/lustre-release
Branch: b2_12
Current Patch Set:
Commit: cfb27a680cd22b993b3ff5e5257154aa3358dbbd

Generated at Sat Feb 10 01:23:29 UTC 2024 using Jira 9.4.14#940014-sha1:734e6822bbf0d45eff9af51f82432957f73aa32c.