[LU-2233] sanityn/40-46 takes ~800-900s - Whamcloud Community JIRA

Details

Type: Improvement
Resolution: Fixed
Priority: Minor
Fix Version/s: Lustre 2.13.0, Lustre 2.12.9
Affects Version/s: None
Labels:
- patch

Rank (Obsolete):
5291

Description

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

Attachments

Issue Links

is related to

LU-10754 sanityn test 47b fails with 'create must fail'

Resolved

Activity

[LU-2233] sanityn/40-46 takes ~800-900s

Gerrit Updater added a comment - 30/Jan/22 3:41 AM

"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

Gerrit Updater added a comment - 30/Jan/22 3:41 AM "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

Gerrit Updater added a comment - 23/Jul/21 9:36 AM

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

Gerrit Updater added a comment - 23/Jul/21 9:36 AM 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

Gerrit Updater added a comment - 12/Feb/21 10:54 PM

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

Gerrit Updater added a comment - 12/Feb/21 10:54 PM 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

Gerrit Updater added a comment - 10/May/19 8:27 PM

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

Gerrit Updater added a comment - 10/May/19 8:27 PM 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

Peter Jones added a comment - 10/May/19 12:47 PM

Landed for 2.13

Peter Jones added a comment - 10/May/19 12:47 PM Landed for 2.13

Gerrit Updater added a comment - 10/May/19 7:12 AM

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

Gerrit Updater added a comment - 10/May/19 7:12 AM 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

Andreas Dilger added a comment - 16/Sep/17 8:34 AM

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

Andreas Dilger added a comment - 16/Sep/17 8:34 AM 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

Alex Zhuravlev added a comment - 26/Oct/12 2:15 AM

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

Alex Zhuravlev added a comment - 26/Oct/12 2:15 AM https://maloo.whamcloud.com/test_sets/adb04eb8-1f33-11e2-bae2-52540035b04c – sanityn took 639s. usually it takes 1300-1700s.

People

Assignee:: Alex Zhuravlev

Reporter:: Alex Zhuravlev

Votes:: 0 Vote for this issue

Watchers:: 4 Start watching this issue

Dates

Created:: 25/Oct/12 1:23 PM

Updated:: 30/Jan/22 3:51 AM

Resolved:: 10/May/19 12:47 PM