https://jira.whamcloud.com This file is an XML representation of an issue en-us 9.4.14 940014 05-12-2023 https://jira.whamcloud.com/browse/LU-10507 Lustre <p>From <a href="https://review.whamcloud.com/#/c/30836/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/#/c/30836/</a></p> <p>{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.<br/> Looking a bit more I see sanity.sh test_27A(), test_65<span class="error">&#91;ij&#93;</span>, test_65m (in case of success), test_406 would also benefit from proper layout restore. The test_65<span class="error">&#91;ij&#93;</span> tests look like they should be combined into a single test so that the layout is always restored, and not/skipped or deleted.<br/> That should be a separate patch so that this one can land more quickly."</p> LU-10507

Use {save,restore}_layout() to save and restore layout in test cases

Improvement Minor Resolved Fixed Jian Yu Jinshan Xiong Fri, 12 Jan 2018 17:32:03 +0000 Mon, 25 Mar 2019 21:31:08 +0000 Fri, 9 Feb 2018 06:01:15 +0000 Lustre 2.11.0 Lustre 2.10.7 0 3 <p>Jinshan Xiong (jinshan.xiong@intel.com) uploaded a new patch: <a href="https://review.whamcloud.com/30858" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/30858</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-10507" title="Use {save,restore}_layout() to save and restore layout in test cases" class="issue-link" data-issue-key="LU-10507"><del>LU-10507</del></a> tests: use </p> {save,restore} <p>_layout() in test<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: ab989121293868716f109ec2a6ed487033a7da37</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="https://review.whamcloud.com/30858/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/30858/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-10507" title="Use {save,restore}_layout() to save and restore layout in test cases" class="issue-link" data-issue-key="LU-10507"><del>LU-10507</del></a> tests: use </p> {save,restore} <p>_layout() in test<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 7b980e101e172d7d8b43a0db2dcaabc8c8c6c855</p> <p>Jian Yu (yujian@whamcloud.com) uploaded a new patch: <a href="https://review.whamcloud.com/34159" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/34159</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-10507" title="Use {save,restore}_layout() to save and restore layout in test cases" class="issue-link" data-issue-key="LU-10507"><del>LU-10507</del></a> tests: use </p> {save,restore} <p>_layout() in test<br/> Project: fs/lustre-release<br/> Branch: b2_10<br/> Current Patch Set: 1<br/> Commit: f02c76ba1141af91720a492100a51e51230d680f</p> <p>Andreas Dilger (adilger@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/34159/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/34159/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-10507" title="Use {save,restore}_layout() to save and restore layout in test cases" class="issue-link" data-issue-key="LU-10507"><del>LU-10507</del></a> tests: use </p> {save,restore} <p>_layout() in test<br/> Project: fs/lustre-release<br/> Branch: b2_10<br/> Current Patch Set: <br/> Commit: dafe2b93e6f4cb67754e960487e92700d6ec279c</p> Related LU-11995 Development Rank 1|hzzr2f: Rank (Obsolete) 9223372036854775807