Whamcloud Community JIRA

Whamcloud Community JIRA https://jira.whamcloud.com This file is an XML representation of an issue en-us

9.4.14

940014

05-12-2023

[LU-3815] Fix sanity-hsm test 34: "Remove file during restore" https://jira.whamcloud.com/browse/LU-3815 Lustre LU-3815

Fix sanity-hsm test 34: "Remove file during restore"

Technical task LU-3647 Critical Resolved Fixed Saurabh Tandan Jodi Levi HSM MB Thu, 22 Aug 2013 13:16:37 +0000 Fri, 22 Dec 2017 10:54:15 +0000 Fri, 4 Nov 2016 15:28:28 +0000 Lustre 2.5.0 Lustre 2.9.0 0 8 patch is at <a href="http://review.whamcloud.com/7431" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/7431</a> We will add tests 34, 35, and 36 to the exception list The above patch is abandoned, need a different patch. I can create a patch in minutes - but it will be conflicted with path 7374 and 7505 which are ready to land, so i tend to think we should land them firstly and then create this one. Or we can land patch 7505 and then I'll merge fix for this issue, along with the fix of <a href="https://jira.whamcloud.com/browse/LU-3761" title="sanity-hsm 55: request on 0x2000013a3:0x2:0x0 is not FAILED" class="issue-link" data-issue-key="LU-3761"><del>LU-3761</del></a> into patch 7374. Adding tests to exception is being handled in <a href="https://jira.whamcloud.com/browse/LU-3561" title="Add a sanity test for HSM" class="issue-link" data-issue-key="LU-3561"><del>LU-3561</del></a> with <a href="http://review.whamcloud.com/#/c/7374/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/#/c/7374/</a> Test cases 34, 35, 36 are trying to delete a file being restored which will never succeed because MDT holds layout lock for restore so unlink is blocked. Therefore I will add them into exception list for now. We may change unlink and rename so that they will acquire LOOKUP only; if this is implemented, we can reenable the test cases again. Jodi, Jinshan, can change <a href="http://review.whamcloud.com/#/c/7374/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/#/c/7374/</a> be tracked by this ticket (and reference it in its Commit-msg!), it makes more sense than leave it linked to <a href="https://jira.whamcloud.com/browse/LU-3561" title="Add a sanity test for HSM" class="issue-link" data-issue-key="LU-3561"><del>LU-3561</del></a>, and this will also allow to close it and be one blocker down ... Also, could it be possible to add sub-tests 40 and 251, actually disabled by the tests environment (TEI-570, and SANITY_HSM_EXCEPT/EXCEPT usage), to the re-freshed exception list ?? This will allow to ask Tools-team to revert their changes for TEI-570 and give us hands-on to re-enable these tests when related issues become fixed. test_251 has already been fixed, but we have to do some more work for test_40. I will add test_40 into exception list if we don't have enough time to work out a patch. In <a href="https://jira.whamcloud.com/browse/LU-3939" title="Test failure on test suite sanity-hsm, subtest test_40" class="issue-link" data-issue-key="LU-3939"><del>LU-3939</del></a> I have a patch to fix test_40, but it is dependent on TEI-570 external change. Thus to verify my patch results I actually need to add a hack to un-exclude test_40 ... So the (John's!) idea is to leave test_40 excluded within your patch and get it land, then after some time ask Tools-team to revert their changes for TEI-570. This will allow us/me to add one Change line in my patch for <a href="https://jira.whamcloud.com/browse/LU-3939" title="Test failure on test suite sanity-hsm, subtest test_40" class="issue-link" data-issue-key="LU-3939"><del>LU-3939</del></a> to remove test_40 from your patch exception list with the fixed test code. Test is still being skipped due to this bug. Reopen until test is running again. The TEI-1041 patch to remove sanity-hsm test_40 and test_251 from the test-framework ALWAYS_EXCEPT list landed two years ago. It should be possible to re-enable those tests by removing them from the sanity-hsm ALWAYS_EXCEPT list and verify that they are now passing. Saurabh, could you please submit a patch to remove test 34/35/36 from the ALWAYS_EXCEPT list in lustre/tests/sanity-hsm.sh and add: <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre>Test-Parameters: testlist=sanity-hsm,sanity-hsm,sanity-hsm,sanity-hsm </pre> </div></div> to the commit message so we have some idea if these tests are now passing. Saurabh Tandan (saurabh.tandan@intel.com) uploaded a new patch: <a href="http://review.whamcloud.com/20079" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/20079</a> Subject: <a href="https://jira.whamcloud.com/browse/LU-3815" title="Fix sanity-hsm test 34: "Remove file during restore"" class="issue-link" data-issue-key="LU-3815"><del>LU-3815</del></a> tests: sanity-hsm - Remove tests from Always_Except Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 9628653a18001f312e2f122a07a86369341fe351 Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/20079/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/20079/</a> Subject: <a href="https://jira.whamcloud.com/browse/LU-3815" title="Fix sanity-hsm test 34: "Remove file during restore"" class="issue-link" data-issue-key="LU-3815"><del>LU-3815</del></a> tests: sanity-hsm - Remove tests from Always_Except Project: fs/lustre-release Branch: master Current Patch Set: Commit: a2009951f9e6c73d29b67c09e00e7057ff7cc973 The patch removing the sanity-hsm tests 34, 35, and 36 has landed to the master branch, 2.8.58. Thus, this ticket can be closed. Duplicate LU-3816 Related Development Rank 1|hzvyvr: Rank (Obsolete) 9858