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-11907 Lustre <p>== sanity test complete, duration 31 sec ============================================================= 14:20:26 (1548926426)<br/> rm: cannot remove '/mnt/lustre/d60g.sanity': Directory not empty<br/> sanity : @@@@@@ FAIL: remove sub-test dirs failed </p> <p>this is because 60g left d60g.sanity in inconsistent state:<br/> striped directory creation failed to create one stripe, then failed to remove direntry (due to same OBD_FAIL_OSD_TXN_START).<br/> when final rmdir fails to load inode with:<br/> LustreError: 25350:0:(llite_lib.c:2414:ll_prep_inode()) new_inode -fatal: rc -2</p> <p>I'm not sure what would be a correct approach here as in the fields we'd suggest to run LFSCK. probably we should use something like lctl rmdir .. to cleanup after 60g?</p> LU-11907

sanity fails to cleanup d60g.sanity

Bug Minor Resolved Fixed Lai Siyao Alex Zhuravlev Thu, 31 Jan 2019 10:15:43 +0000 Sat, 4 Jan 2020 00:03:46 +0000 Wed, 29 May 2019 13:00:41 +0000 Lustre 2.12.2 Lustre 2.12.3 Lustre 2.13.0 Lustre 2.12.4 0 5 <p>While fixing this with LFSCK should hopefully work, it is probably my least-preferred option, next least preferred option is using "lfs rmentry" (or whatever it is called).  </p> <p>The reason I don't like those is because this isn't what users will use, and they may not even know they exist. They will use "rmdir" or "rm -r", and that should work. The main reason we added "lfs rmentry" is to allow deleting entries on a remote MDT when the MDT is permanently lost. Normally, we don't want to allow a user to delete a remote directory when the MDT is just temporarily offline, because it may be full of files. </p> <p>right, I think this is a special case. I actually doubt such an outcome was designed. though another concern is that the test leaves the filesystem inconsistent and if we begin to run fsck/lfsck at some point (iirc, we discussed this possibility) then that would report an error.<br/> another thought was that it's probably OK to let users to access/remove striped directories with missing stripes? currently this is impossible, AFAIU.</p> <p>Yes, being able to read/unlink directories with missing stripes is very useful. We can't allow new files to be created therein, but we might consider to allow the entries to be migrated to another MDT (probably as a separate patch, if it is more complex than just allowing read/unlink). </p> <p>Yes, I agree. After we support access to directories with missing stripes, migrate such directory will be similar to migrate directory with bad hash.</p> <p>Lai Siyao (lai.siyao@whamcloud.com) uploaded a new patch: <a href="https://review.whamcloud.com/34750" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/34750</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-11907" title="sanity fails to cleanup d60g.sanity" class="issue-link" data-issue-key="LU-11907"><del>LU-11907</del></a> dne: allow access to striped dir with broken layout<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 1717d19d1eefd6be6dcc78ea55283ceccc68e920</p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/34750/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/34750/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-11907" title="sanity fails to cleanup d60g.sanity" class="issue-link" data-issue-key="LU-11907"><del>LU-11907</del></a> dne: allow access to striped dir with broken layout<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: d2725563e7afa17a41a53aa65255a31380606d23</p> <p>Landed for 2.13</p> <p>Minh Diep (mdiep@whamcloud.com) uploaded a new patch: <a href="https://review.whamcloud.com/36939" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/36939</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-11907" title="sanity fails to cleanup d60g.sanity" class="issue-link" data-issue-key="LU-11907"><del>LU-11907</del></a> dne: allow access to striped dir with broken layout<br/> Project: fs/lustre-release<br/> Branch: b2_12<br/> Current Patch Set: 1<br/> Commit: a7ddb9c9e20a1593b46fd16862c4bfaea66c0f41</p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/36939/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/36939/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-11907" title="sanity fails to cleanup d60g.sanity" class="issue-link" data-issue-key="LU-11907"><del>LU-11907</del></a> dne: allow access to striped dir with broken layout<br/> Project: fs/lustre-release<br/> Branch: b2_12<br/> Current Patch Set: <br/> Commit: 5b1ea58c21edd17c2cb1f4ecdbbeb5bbdaa1444b</p> Related LU-11681 Development Rank 1|i00an3: Rank (Obsolete) 9223372036854775807 Severity