[LU-11820] reduce conf-sanity test duration Created: 20/Dec/18  Updated: 18/Jun/21  Resolved: 04/Jan/19

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

Type: Improvement Priority: Minor
Reporter: Andreas Dilger Assignee: Andreas Dilger
Resolution: Fixed Votes: 0
Labels: None

Issue Links:
Related
is related to LU-14772 split conf-sanity into 2 or 3 parts In Progress
Rank (Obsolete): 9223372036854775807

 Description   

Currently, conf-sanity.sh takes about 4h to finish for review-zfs, 5.75h for review-dne-part-3, and 6.75h for review-dne-zfs-part-3, which is too long.

Test 106 in regular conf-sanity takes about 40 minutes, and should be disabled for SLOW=no. It could possibly be shortened by running fewer loops.

Test 32 is also taking a relatively long time (each one about 100s), but it is prudent to continue running upgrades from multiple test images to ensure compatibility.

One area I think that can be improved significantly is to avoid formatting and reformatting the filesystem for each test. Some of the conf-sanity tests assume that a filesystem is available, but many of the tests want to format a filesystem to their own specifications, then reformat the filesystem back to the "default" configuration and remount it at the end of the test, only to unmount and format again for the next test. I think it would make sense for conf-sanity tests to assume the filesystem is not mounted at the beginning (or maybe tests after a certain number), and skip the "reformat back to default and remount" step at the end. This should only be done once at the end of conf-sanity to return the filesystem back to a known state. This would save a couple minutes for each subtest, maybe as much as 2h for each test run.



 Comments   
Comment by Gerrit Updater [ 20/Dec/18 ]

Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/33902
Subject: LU-11820 tests: skip conf-sanity test_106 for SLOW=no
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 81c33655dc5ff3173b7a71e52dcba003cf99baa5

Comment by Gerrit Updater [ 04/Jan/19 ]

Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/33902/
Subject: LU-11820 tests: skip conf-sanity test_106 for SLOW=no
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 8520cfb2f77b6fc89a9c0e174f783d745dbaf2e3

Comment by Peter Jones [ 04/Jan/19 ]

Landed for 2.13

Generated at Sat Feb 10 02:47:13 UTC 2024 using Jira 9.4.14#940014-sha1:734e6822bbf0d45eff9af51f82432957f73aa32c.