[LU-11330] replay-single test_70d: Directory not empty rmdir fails

• Status: Resolved
• Project: Lustre
• Affects Version/s: Lustre 2.12.0
• Fix Version/s: Lustre 2.13.0, Lustre 2.12.1
• Type: Bug
• Priority: Major
• Reporter: Maloo
• Assignee: Alex Zhuravlev
• Resolution: Fixed
• Severity: 3

Description

This issue was created by maloo for Andreas Dilger <[e-mail]>

This issue relates to the following test suite run: https://testing.whamcloud.com/test_sets/0b9dcc0a-adf2-11e8-bd05-52540065bddc

test_70d failed with the following error:

Started 7681
rm: cannot remove '/mnt/lustre/d70d.replay-single/test1': Directory not empty
rmdir fails
7681 stopped

VVVVVVV DO NOT REMOVE LINES BELOW, Added by Maloo for auto-association VVVVVVV
replay-single test_70d - 7681 stopped

Issue Links

• is related to: LU-9157 replay-single test_80c: rmdir failed
• is related to: LU-12068 sanity-lfsck test_6b: (7.2) 0x0 is no...
• is related to: LU-11366 replay-single timeout test 80f: rm: c...

Comments

[Alex Zhuravlev]

I'm able to reproduce this w/o any failover, just with this:

for ((i = 0; i < 5000; i++)); do
        $LFS mkdir -i0 -c2 $DIR/$tdir/test || error mkdir failed"
        touch $DIR/$tdir/test/a || error "rmdir failed"
        mkdir $DIR/$tdir/test/b || error "mkdir failed"
        rm -rf $DIR/$tdir/test || error "rmdir failed"
done

the root cause is that sometimes hash of "a" or "b" can be 0, so during readdir MDT returns "." (hard coded hash=1), then ".." (hash=2) and then all the remaining real entries including that one with hash=0. and this confuses the client.

IMO, it deserves to be "major" issue

[Gerrit Updater]

Alex Zhuravlev ([e-mail]) uploaded a new patch: https://review.whamcloud.com/34098
Subject: LU-11330 osd-zfs: hash for . and .. must not be a positive number
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: fe4a0c155179bc3810f9bd449539b147a943daa4

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/34098/
Subject: LU-11330 osd-zfs: hash for ./.. must be 0
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: fb75af7d45d1217c877f75c4296f9df0cc731604

[Peter Jones]

Landed for 2.13

[Gerrit Updater]

Minh Diep ([e-mail]) uploaded a new patch: https://review.whamcloud.com/34451
Subject: LU-11330 osd-zfs: hash for ./.. must be 0
Project: fs/lustre-release
Branch: b2_12
Current Patch Set: 1
Commit: 74b0f9d6ee3278084f5f8f52d1ea3fb35eb4d009

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/34451/
Subject: LU-11330 osd-zfs: hash for ./.. must be 0
Project: fs/lustre-release
Branch: b2_12
Current Patch Set:
Commit: 945db23fad2bb38a2a4ebaec9407f6ae20ed3dc7