[LU-10507] Use {save,restore}_layout() to save and restore layout in test cases

• Status: Resolved
• Project: Lustre
• Fix Version/s: Lustre 2.11.0, Lustre 2.10.7
• Type: Improvement
• Priority: Minor
• Reporter: Jinshan Xiong (Inactive)
• Assignee: Jian Yu
• Resolution: Fixed

Description

From https://review.whamcloud.com/#/c/30836/

{save,restore}_layout() are "probably more reliable than the mechanism used in e.g. sanity-pfl.sh test_10, which tries to parse the layout parameters and restore them, but that will only work for simple layouts. It would be good to update that test to use save_layout() and restore_layout() as well.
Looking a bit more I see sanity.sh test_27A(), test_65[ij], test_65m (in case of success), test_406 would also benefit from proper layout restore. The test_65[ij] tests look like they should be combined into a single test so that the layout is always restored, and not/skipped or deleted.
That should be a separate patch so that this one can land more quickly."

Issue Links

• is related to: LU-11995 sanity test_65n returned 127

Comments

[Gerrit Updater]

Jinshan Xiong ([e-mail]) uploaded a new patch: https://review.whamcloud.com/30858
Subject: LU-10507 tests: use

{save,restore}

_layout() in test
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: ab989121293868716f109ec2a6ed487033a7da37

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/30858/
Subject: LU-10507 tests: use

{save,restore}

_layout() in test
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 7b980e101e172d7d8b43a0db2dcaabc8c8c6c855

[Gerrit Updater]

Jian Yu ([e-mail]) uploaded a new patch: https://review.whamcloud.com/34159
Subject: LU-10507 tests: use

{save,restore}

_layout() in test
Project: fs/lustre-release
Branch: b2_10
Current Patch Set: 1
Commit: f02c76ba1141af91720a492100a51e51230d680f

[Gerrit Updater]

Andreas Dilger ([e-mail]) merged in patch https://review.whamcloud.com/34159/
Subject: LU-10507 tests: use

{save,restore}

_layout() in test
Project: fs/lustre-release
Branch: b2_10
Current Patch Set:
Commit: dafe2b93e6f4cb67754e960487e92700d6ec279c