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-6225 Lustre <p> if a test fails with an error, we just exit:</p> <div class="code panel" style="border-width: 1px;"><div class="codeContent panelContent"> <pre class="code-java">error() { error_noexit <span class="code-quote">"$@"</span> exit 1 } </pre> </div></div> <p>and therefore the end of run_one() does not execute, what means we do not cleanup the env settings (e.g. fail_loc) before running the next test.</p> LU-6225

test-framework does not cleanup the env for failed tests

Bug Major Resolved Fixed WC Triage gaurav mahajan Mon, 9 Feb 2015 11:51:29 +0000 Mon, 29 May 2017 05:32:49 +0000 Thu, 4 Feb 2016 05:31:40 +0000 Lustre 2.5.3 Lustre 2.8.0 Lustre 2.8.0 0 8 <p>Gaurav Mahajan (gaurav.mahajan@seagate.com) uploaded a new patch: <a href="http://review.whamcloud.com/13689" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/13689</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6225" title="test-framework does not cleanup the env for failed tests" class="issue-link" data-issue-key="LU-6225"><del>LU-6225</del></a> test: test-framework does not cleanup for failed tests<br/> Project: fs/lustre-release<br/> Branch: b2_5<br/> Current Patch Set: 1<br/> Commit: 05ef6f49b13abbfd33c7402bf514440e0315c3c0</p> <p>Does this issue only affect b2_5 or is it relevant for master too?</p> <p>Andreas Dilger (andreas.dilger@intel.com) uploaded a new patch: <a href="http://review.whamcloud.com/13692" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/13692</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6225" title="test-framework does not cleanup the env for failed tests" class="issue-link" data-issue-key="LU-6225"><del>LU-6225</del></a> test: test-framework does not cleanup for failed tests<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 00afee36ddfb1fb7bb4b58ca82b0fef41f7f4527</p> <p>The tests which call error_noexit() will return with a value (non zero value) to run_one(). Then run_one() will call error() in case of non zero return value from the test which will again call error_noexit(). </p> <p>If we include reset_fail_loc to error_noexit () then it becomes kind of doing the things which are already done as error() will ultimately get called incase of non zero return value (ie on error)</p> <p>In case of error_and_remount(), agree that it calls error_no_exit() and exits from there. I guess modification is needed for test framework to handle this looping of error() and error_no_exit().</p> <p>As of now, I will accept the comments and move reset_fail_loc to error_no_exit(). But modifications are needed for test_framework.sh </p> <p>Please leave your comments on this.</p> <p>Hit in interop testing <br/> <a href="https://testing.hpdd.intel.com/test_sessions/2b6be11e-9033-11e5-a833-5254006e85c2" class="external-link" target="_blank" rel="nofollow noopener">https://testing.hpdd.intel.com/test_sessions/2b6be11e-9033-11e5-a833-5254006e85c2</a><br/> server: lustre-master/3251 EL7 <br/> client 2.7.0</p> <p>Server: Master, Build# 3266, Tag 2.7.64 , RHEL 7 <br/> Client: 2.5.5, b2_5_fe/62<br/> <a href="https://testing.hpdd.intel.com/test_sets/0d665f82-a000-11e5-a33d-5254006e85c2" class="external-link" target="_blank" rel="nofollow noopener">https://testing.hpdd.intel.com/test_sets/0d665f82-a000-11e5-a33d-5254006e85c2</a><br/> <a href="https://testing.hpdd.intel.com/test_sets/0d289292-a000-11e5-a33d-5254006e85c2" class="external-link" target="_blank" rel="nofollow noopener">https://testing.hpdd.intel.com/test_sets/0d289292-a000-11e5-a33d-5254006e85c2</a></p> <p>Another instance found for :<br/> Server: Master , Build# 3276<br/> Client: b2_7_fe/34<br/> <a href="https://testing.hpdd.intel.com/test_sets/1c4ee028-a5a5-11e5-a14a-5254006e85c2" class="external-link" target="_blank" rel="nofollow noopener">https://testing.hpdd.intel.com/test_sets/1c4ee028-a5a5-11e5-a14a-5254006e85c2</a><br/> <a href="https://testing.hpdd.intel.com/test_sets/1c77ef4a-a5a5-11e5-a14a-5254006e85c2" class="external-link" target="_blank" rel="nofollow noopener">https://testing.hpdd.intel.com/test_sets/1c77ef4a-a5a5-11e5-a14a-5254006e85c2</a></p> <p>Another Instance for interop: EL7 Server/2.7.1 Client<br/> Server: master, build# 3303, RHEL 7 <br/> Client: 2.7.1, b2_7_fe/34<br/> <a href="https://testing.hpdd.intel.com/test_sets/7a715f8a-bac4-11e5-9137-5254006e85c2" class="external-link" target="_blank" rel="nofollow noopener">https://testing.hpdd.intel.com/test_sets/7a715f8a-bac4-11e5-9137-5254006e85c2</a><br/> <a href="https://testing.hpdd.intel.com/test_sets/7a2d8fe4-bac4-11e5-9137-5254006e85c2" class="external-link" target="_blank" rel="nofollow noopener">https://testing.hpdd.intel.com/test_sets/7a2d8fe4-bac4-11e5-9137-5254006e85c2</a></p> <p>Encountered same issue for tag 2.7.66 for FULL- EL7.1 Server/EL6.7 Client , master , build# 3314<br/> <a href="https://testing.hpdd.intel.com/test_sets/924b1b78-ca91-11e5-9609-5254006e85c2" class="external-link" target="_blank" rel="nofollow noopener">https://testing.hpdd.intel.com/test_sets/924b1b78-ca91-11e5-9609-5254006e85c2</a></p> <p>Another failure for master : Tag 2.7.66 FULL - EL7.1 Server/SLES11 SP3 Client, build# 3314<br/> <a href="https://testing.hpdd.intel.com/test_sets/b0f05d56-ca7b-11e5-9609-5254006e85c2" class="external-link" target="_blank" rel="nofollow noopener">https://testing.hpdd.intel.com/test_sets/b0f05d56-ca7b-11e5-9609-5254006e85c2</a></p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/13692/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/13692/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6225" title="test-framework does not cleanup the env for failed tests" class="issue-link" data-issue-key="LU-6225"><del>LU-6225</del></a> test: test-framework does not cleanup for failed tests<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: dc3a6a13d2cec51ca5d8ab1c870ce4cee3d80494</p> <p>Landed for 2.8</p> Duplicate LU-2312 Development Epic/Theme patch test Project Rank 1|hzx5y7: Rank (Obsolete) 17420 Severity