[LU-15235] sanity test_56od: '/usr/bin/lfs find /mnt/lustre/d56od.sanity -newerbb /mnt/lustre/f56od.sanity.ref ! -newerbb /mnt/lustre/f56od.sanity.negref -type d' wrong: found 5, expected 4 Created: 16/Nov/21 Updated: 24/Aug/23 Resolved: 24/Aug/23 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | Lustre 2.15.3 |
| Fix Version/s: | Lustre 2.16.0 |
| Type: | Bug | Priority: | Minor |
| Reporter: | Maloo | Assignee: | Andreas Dilger |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||
| Severity: | 3 | ||||
| Rank (Obsolete): | 9223372036854775807 | ||||
| Description |
|
This issue was created by maloo for S Buisson <sbuisson@ddn.com> This issue relates to the following test suite run: https://testing.whamcloud.com/test_sets/730cff32-7cca-48db-b3a5-45e1abf75c4e test_56od failed with the following error: '/usr/bin/lfs find /mnt/lustre/d56od.sanity -newerbb /mnt/lustre/f56od.sanity.ref ! -newerbb /mnt/lustre/f56od.sanity.negref -type d' wrong: found 5, expected 4 VVVVVVV DO NOT REMOVE LINES BELOW, Added by Maloo for auto-association VVVVVVV |
| Comments |
| Comment by Cliff White (Inactive) [ 19/Apr/22 ] |
|
Appear to have hit this again in 2.15 testing. Not much data available in the test logs, is this a possible clean-up issue? |
| Comment by Andreas Dilger [ 27/Oct/22 ] |
|
This test is exercising functionality that does not exist in older branches. The patch was landed to master in commit v2_13_53-145-g186b97e68a, and never ported to b2_12. It looks like it has an appropriate MDS version check, but it looks like the "btime_supported" check is broken in some way so it is still being run against unsupported servers (i.e. 2.12.9 or 2.10.8): It looks like the "btime_supported" function is only verifying that the client's "lfs find" command has the -btime option, and not really checking if it returns something useful, so I think that whole function should be removed and just stick with the MDS version check. I don't think the client version check is useful either at this point, since any client that is running this test will have this patch, so it can also be removed. Also note that test_56oc was "correctly" skipping old MDS versions because it was trying to run the btime_supported() function that was defined after test_56oc, so this would always return false and depend only on the version check. That bug-that-is-fixing-the-test will also be addressed by removing this check from test_56oc. |
| Comment by Gerrit Updater [ 05/Jul/23 ] |
|
"Andreas Dilger <adilger@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/51580 |
| Comment by Gerrit Updater [ 24/Aug/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/51580/ |
| Comment by Peter Jones [ 24/Aug/23 ] |
|
Landed for 2.16 |