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-2352 Lustre <p>Numerous tests depend on the debugfs/dumpe2fs/etc utilities and assume the fs type is ldiskfs. For the moment all of these tests are skipped for other fs types. However, they all need to be updated (if possible) to run using the correct utility for that fs type. For zfs filesystems most things can be accomplished using zdb.</p> LU-2352

Tests must use DEBUGFS or ZDB as appropriate

Task Minor Closed Duplicate Li Wei Brian Behlendorf zfs Thu, 6 Oct 2011 16:29:39 +0000 Wed, 20 Mar 2013 13:38:31 +0000 Wed, 20 Mar 2013 13:38:16 +0000 Lustre 2.4.0 0 4 <p>Some of this work may depend on getting ORI-160 finished so objects can be referenced by name.</p> <p>Some tests use debugfs command "unlink", "rm", "write", etc. I didn't find how zdb could do such modifications. Also, zdb takes only object numbers, not path names. So, I have to change the tests to mount back ends if possible.</p> <p>I'd like to hear some advices on how to deal with conf-sanity 38. I tried mounting the target with the back end file system and unlink "lov_objid", but it didn't work. Both ldiskfs and ZFS OSD insert a FID_SEQ_LOCAL_FILE object into the object index as well as the root directory. Unlinking the object from the root directory thus results in a) EEXIST when recreating the object in the ZFS case or b) a nonexistent inode referenced by the object index in the ldiskfs case, which causes an error too. An immediate solution is to remove the object from the object index as well as the root directory. That should be easy to do in the ZFS case, but might require some work in the ldiskfs case. Another solution, at the risk of "adapting Lustre to tests", is to stop inserting local objects into the object index. Stepping back a little, I wonder how lov_objid become lost or zeroed in real world and if the test is not so useful that can be ignored.</p> <p>When reviewing ORI-162, it is my opinion that the FID_SEQ_LOCAL_FILE objects should not be in the OI, but rather be handled via by-name lookup only. That is no more costly than by-FID lookup, since both require a single ZAP lookup, and it may be considerably faster for these frequently-used object since the root ZAP is small and the OI will be huge.</p> <p>This also avoids the problem seen here where any "by hand" modification of the underlying filesystem (which was the whole reason to make the ZFS OSD ZPL compatible) would cause the filesystem to break.</p> <p>As for reasons why this may be needed in real life:</p> <ul class="alternate" type="square"> <li>if an OST is lost/corrupted and a new OST is formatted with the same index it is necessary to reset the lov_objids file</li> <li>if the lov_objids file is itself corrupted, being able to simply delete it and have Lustre recreate it from the OST LAST_ID data is very convenient (the only loss being a few orphaned zero-length objects on the OSTs)</li> <li>similarly, truncating or deleting the last_rcvd file to handle cases where it is corrupted (either by storage or software bug like duplicate export UUIDs), and then having Lustre recreate it is very convenient</li> <li>being able to modify the OST UUID in last_rcvd is also important on occasion</li> <li>sometimes corruption of LAST_ID needs manual repair</li> </ul> <p>Ideally, these situations would all be handled automatically without need for user interaction, but that is not the case today. In most cases there <em>could</em> be automatic recovery (e.g. rebuild LAST_ID by scanning the O/</p> {seq} <p>/d* directories for the highest OID), but this also needs development effort, and isn't always possible to handle automatically (e.g. the OST doesn't know what is in the lov_objids file on the MDT, and getting this wrong can result in more filesystem corruption).</p> <p>Li Wei, this issue with inserting local files into OI exists also on master, so it is not Orion issue. I see Fan Yong removed that in recent patches related to lfsck work though it is not enough. We discussed this with Alex day ago also, the problem is wider than seems. I tend to think we need separate bug about how to store local files and their index. Refer to <a href="https://jira.whamcloud.com/browse/LU-2340" title="Storing server local files" class="issue-link" data-issue-key="LU-2340"><del>ORI-617</del></a></p> <p>Since there's actually no ZFS equivalent of debugfs, I think this can be closed as a dup of <a href="https://jira.whamcloud.com/browse/LU-2353" title="Tests mount servers as ldiskfs" class="issue-link" data-issue-key="LU-2353"><del>LU-2353</del></a>.</p> Development Epic server Project Rank 1|hzuv9j: Rank (Obsolete) 2727 Release Version History Story Points 2.0