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-4690 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="http://maloo.whamcloud.com/test_sets/0dd7df32-a0b5-11e3-9f3a-52540035b04c" class="external-link" target="_blank" rel="nofollow noopener">http://maloo.whamcloud.com/test_sets/0dd7df32-a0b5-11e3-9f3a-52540035b04c</a>.</p> <p>The sub-test test_4 failed with the following error:</p> <blockquote> <p>Expect error removing in-use dir /mnt/lustre/remote_dir</p></blockquote> <p>There have been many of these lately. maloo reports:</p> <p>Failure Rate: 31.00% of last 100 executions <span class="error">&#91;all branches&#93;</span></p> <p>A search shows failures like this started happening 2/26, none earlier.<br/> Suspect something landed just recently causing these failures.</p> <p>Info required for matching: sanity 4</p> LU-4690

sanity test_4: Expect error removing in-use dir /mnt/lustre/remote_dir

Bug Blocker Resolved Fixed Di Wang Maloo dne2 Fri, 28 Feb 2014 21:39:19 +0000 Wed, 13 Jul 2016 23:56:09 +0000 Mon, 12 May 2014 13:04:42 +0000 Lustre 2.6.0 Lustre 2.5.1 Lustre 2.5.2 Lustre 2.6.0 Lustre 2.5.2 0 11 <p>I see that test_4 was only added to sanity.sh very recently, in commit 1f55f2a9071d5e7db4042b959723086dee1c379a, <a href="https://jira.whamcloud.com/browse/LU-4471" title="Failed &#39;rmdir&#39; on remote directories still removes directory on MDT0" class="issue-link" data-issue-key="LU-4471"><del>LU-4471</del></a> mdd: mdd_unlink: do trans_start after sanity check.</p> <p>That would explain why all failures are recent. Strongly suspect something is very wrong with this new subtest.</p> <p>I note that this change was back ported into b2_5 at nearly the same time. I'll bet it's causing test failures there too.</p> <p>Hmm, right now we will check whether the object is empty during getattr. But it is missing for striped dir. And also current empty check for remote directory is kind of jacky, given that we already have remote object iteration from LFSCK. I will check what I can do here. And in the mean time, I would suggest to disable the test. </p> <p>Disable <a href="https://jira.whamcloud.com/browse/LU-4690" title="sanity test_4: Expect error removing in-use dir /mnt/lustre/remote_dir" class="issue-link" data-issue-key="LU-4690"><del>LU-4690</del></a> temporarily. <a href="http://review.whamcloud.com/#/c/9440/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/#/c/9440/</a></p> <p> <a href="http://review.whamcloud.com/9511" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/9511</a> </p> <p>I have retriggered Change, 9511 test.</p> <p>Does this test need to be re-enabled now that Change, 9511 has landed? Or can this ticket be closed?</p> <p>There is another patch <a href="http://review.whamcloud.com/#/c/10261/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/#/c/10261/</a> to cleanup the 9511 a bit. Probably close the ticket after 10261 is landed?</p> <p>Patches landed to Master.</p> <p>Looks the patch caused serious problem with quota:</p> <p>In lod_attr_set():</p> <div class="code panel" style="border-width: 1px;"><div class="codeContent panelContent"> <pre class="code-java"> <span class="code-keyword">for</span> (i = 0; i &lt; lo-&gt;ldo_stripenr; i++) { LASSERT(lo-&gt;ldo_stripe[i]); + <span class="code-keyword">if</span> (dt_object_exists(lo-&gt;ldo_stripe[i]) == 0) + <span class="code-keyword">continue</span>; rc = dt_attr_set(env, lo-&gt;ldo_stripe[i], attr, handle, capa); </pre> </div></div> <p>I think dt_object_exists() will always return false for osp object, then any chown/chgrp will never be applied to OST objects. Any thoughts?</p> <p>Because test_34 of s-q was disabled for <a href="https://jira.whamcloud.com/browse/LU-4515" title="Test failure sanity-quota test_34: Used space for user 60000 is 0, expected 2048" class="issue-link" data-issue-key="LU-4515"><del>LU-4515</del></a>, this problem wasn't found by maloo test, I've submit a patch to re-enable test_34 in <a href="https://jira.whamcloud.com/browse/LU-4515" title="Test failure sanity-quota test_34: Used space for user 60000 is 0, expected 2048" class="issue-link" data-issue-key="LU-4515"><del>LU-4515</del></a>.</p> <p>I think dt_object_exists() will always return false for osp object, then any chown/chgrp will never be applied to OST objects. Any thoughts?</p> <p>Hmm, we actually separate the remote and exist flag for MD OSP object, but for OST object, this check might have problem. The easiest fix might be add S_ISDIR(dt-&gt;do_lu.lo_header-&gt;loh_attr) check here? </p> <p>Thank you, Di. I'll try to compose a patch according to your comment and track the fix in <a href="https://jira.whamcloud.com/browse/LU-5296" title="lod_attr_set() skips attr_set on osp objects incorrectly" class="issue-link" data-issue-key="LU-5296"><del>LU-5296</del></a>.</p> Related LU-4471 LU-4215 LU-3696 LU-5296 LU-5675 LU-5016 Development Rank 1|hzwgdr: Rank (Obsolete) 12888 Severity