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-10199 Lustre <p>This issue was created by maloo for Bob Glossman &lt;bob.glossman@intel.com&gt;</p> <p>This issue relates to the following test suite run: <a href="https://testing.hpdd.intel.com/test_sets/740cdf26-c31d-11e7-88ab-52540065bddc" class="external-link" target="_blank" rel="nofollow noopener">https://testing.hpdd.intel.com/test_sets/740cdf26-c31d-11e7-88ab-52540065bddc</a>.</p> <p>The sub-test test_56xb failed with the following error:</p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre>hard links individually migrated (12 != 11) </pre> </div></div> <p>strongly suspect this is due to the very recently landed "<a href="https://jira.whamcloud.com/browse/LU-6051" title="&quot;lfs_migrate&quot; improvements" class="issue-link" data-issue-key="LU-6051"><del>LU-6051</del></a> utils: allow lfs_migrate to handle hard links"<br/> test 56xb is brand new, only introduced in this patch.<br/> I'm wondering if this test has ever been passed on any SLES version.</p> <p>Info required for matching: sanity 56xb</p> LU-10199

sanity test_56xb: hard links individually migrated (12 != 11)

Bug Major Resolved Fixed Steve Guminski Maloo Mon, 6 Nov 2017 18:17:14 +0000 Mon, 9 Apr 2018 22:22:29 +0000 Mon, 26 Mar 2018 01:00:19 +0000 Lustre 2.11.0 Lustre 2.11.0 Lustre 2.10.4 0 6 <p>Steve Guminski (stephenx.guminski@intel.com) uploaded a new patch: <a href="https://review.whamcloud.com/29954" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/29954</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-10199" title="sanity test_56xb: hard links individually migrated (12 != 11)" class="issue-link" data-issue-key="LU-10199"><del>LU-10199</del></a> utils: revert changeset 25851 due to sanity fail<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: e1c26a24dd6ace3e68e5b66c714eb13ec54a2875</p> <p>Do we have any idea why this is failing? I'd prefer a patch that fixes a problem vs. reverting it. Next best would be a patch to skip <em>just</em> the failing test for SLES, so that we keep the patch landed and testing under RHEL, and give us some time to fix the SLES issue.</p> <p>I just reran the sanity test on my local RHEL system, and it fails there, too.  So I'm not sure what has changed between the time that the patch was originally submitted and now that would cause this change in behavior.  I'll submit a new patch that disables the test instead of reverting.</p> <p>Here is an instance of it failing on el7 too:<br/> <a href="https://testing.hpdd.intel.com/test_sets/216f1a9c-c361-11e7-a943-52540065bddc" class="external-link" target="_blank" rel="nofollow noopener">https://testing.hpdd.intel.com/test_sets/216f1a9c-c361-11e7-a943-52540065bddc</a></p> <p>Is it possible that it originally passed test on el7.3 and is now failing on el7.4?</p> <p>Yes, that's possible.  My local CentOS test system was upgraded during that period.</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="https://review.whamcloud.com/29954/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/29954/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-10199" title="sanity test_56xb: hard links individually migrated (12 != 11)" class="issue-link" data-issue-key="LU-10199"><del>LU-10199</del></a> utils: disable sanity test 56xb<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 3ac484707aecb97ab8aa4716110205108b0535ea</p> <p>Steve Guminski (stephenx.guminski@intel.com) uploaded a new patch: <a href="https://review.whamcloud.com/30013" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/30013</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-10199" title="sanity test_56xb: hard links individually migrated (12 != 11)" class="issue-link" data-issue-key="LU-10199"><del>LU-10199</del></a> lfs: Fix file name comparison in lfs_migrate<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 3a7fd98d46729b0d430b6fe30c55cba275574303</p> <p>It would be good to have some examples of how this is broken, since I can't imagine how "lfs fid2path" is generating a non-absolute pathnames.</p> <p>Ben Evans (bevans@cray.com) uploaded a new patch: <a href="https://review.whamcloud.com/30078" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/30078</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-10199" title="sanity test_56xb: hard links individually migrated (12 != 11)" class="issue-link" data-issue-key="LU-10199"><del>LU-10199</del></a> utils: strip // from fid2path<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 5a87fb56b121fdcd623cb9298e53903ee5719ff1</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="https://review.whamcloud.com/30078/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/30078/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-10199" title="sanity test_56xb: hard links individually migrated (12 != 11)" class="issue-link" data-issue-key="LU-10199"><del>LU-10199</del></a> utils: strip // from fid2path<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 116b73d69bf2462fa9e9b8b9f629794bbda90ee5</p> <p>Landed for 2.11</p> <p>Steve Guminski (stephenx.guminski@intel.com) uploaded a new patch: <a href="https://review.whamcloud.com/30616" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/30616</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-10199" title="sanity test_56xb: hard links individually migrated (12 != 11)" class="issue-link" data-issue-key="LU-10199"><del>LU-10199</del></a> tests: Re-enable sanity test 56xb<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 14c03c1e43f62fa79fc38288834ebc0a5df40cf2</p> <p>The patch to re-enable this test was not landed, and it is still being skipped.</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="https://review.whamcloud.com/30616/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/30616/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-10199" title="sanity test_56xb: hard links individually migrated (12 != 11)" class="issue-link" data-issue-key="LU-10199"><del>LU-10199</del></a> tests: Re-enable sanity test 56xb<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: d90d9cd45e977d1672272786cb74d4941b6604a2</p> <p>more seen on b2_10:<br/> <a href="https://testing.hpdd.intel.com/test_sets/62af78a4-2fa8-11e8-b3c6-52540065bddc" class="external-link" target="_blank" rel="nofollow noopener">https://testing.hpdd.intel.com/test_sets/62af78a4-2fa8-11e8-b3c6-52540065bddc</a><br/> <a href="https://testing.hpdd.intel.com/test_sets/9ff156a6-2fa8-11e8-b3c6-52540065bddc" class="external-link" target="_blank" rel="nofollow noopener">https://testing.hpdd.intel.com/test_sets/9ff156a6-2fa8-11e8-b3c6-52540065bddc</a><br/> <a href="https://testing.hpdd.intel.com/test_sets/62af78a4-2fa8-11e8-b3c6-52540065bddc" class="external-link" target="_blank" rel="nofollow noopener">https://testing.hpdd.intel.com/test_sets/62af78a4-2fa8-11e8-b3c6-52540065bddc</a><br/> <a href="https://testing.hpdd.intel.com/test_sets/f0f35630-2fda-11e8-b3c6-52540065bddc" class="external-link" target="_blank" rel="nofollow noopener">https://testing.hpdd.intel.com/test_sets/f0f35630-2fda-11e8-b3c6-52540065bddc</a><br/> <a href="https://testing.hpdd.intel.com/test_sets/f58874a0-306b-11e8-b74b-52540065bddc" class="external-link" target="_blank" rel="nofollow noopener">https://testing.hpdd.intel.com/test_sets/f58874a0-306b-11e8-b74b-52540065bddc</a></p> <p>This problem is not fixed, only the test is in ALWAYS_EXCEPT on master. </p> <p>As a general rule, any ticket with the <tt>always_except</tt> label should not be closed. This label should be removed when the patch that re-enables the test is landed. I've often found, as in this case, that a ticket is marked resolved, but the reason the test is no longer failing is because the test has been disabled. </p> <p>this test was fixed on master (probably) by <a href="https://review.whamcloud.com/30078" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/30078</a> and then reenabled by <a href="https://review.whamcloud.com/30616" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/30616</a>.</p> <p>Sorry, I was looking at an old branch that still showed 56xb as being in ALWAYS_EXCEPT. </p> <p>Bob Glossman (bob.glossman@intel.com) uploaded a new patch: <a href="https://review.whamcloud.com/31775" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/31775</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-10199" title="sanity test_56xb: hard links individually migrated (12 != 11)" class="issue-link" data-issue-key="LU-10199"><del>LU-10199</del></a> utils: strip // from fid2path<br/> Project: fs/lustre-release<br/> Branch: b2_10<br/> Current Patch Set: 1<br/> Commit: 2fa1e1bf7a7e6f9c60fd130dfcb59b168f9b5de2</p> <p>John L. Hammond (john.hammond@intel.com) merged in patch <a href="https://review.whamcloud.com/31775/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/31775/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-10199" title="sanity test_56xb: hard links individually migrated (12 != 11)" class="issue-link" data-issue-key="LU-10199"><del>LU-10199</del></a> utils: strip // from fid2path<br/> Project: fs/lustre-release<br/> Branch: b2_10<br/> Current Patch Set: <br/> Commit: a2b9a5dabf5a6a66192cc15beb03fba8bde167af</p> Development Rank 1|hzzn73: Rank (Obsolete) 9223372036854775807 Severity