Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ceph_test_libcephfs: tolerate duplicated entries in readdir #7246

Merged
merged 1 commit into from Jan 20, 2016

Conversation

ukernel
Copy link
Contributor

@ukernel ukernel commented Jan 15, 2016

If a dirfrag get fragmented in the middle of readdir, the readdir
result may contain duplicated entries. (but there shouldn't be
any missing entries)

Fixes: #14377
Signed-off-by: Yan, Zheng zyan@redhat.com

If a dirfrag get fragmented in the middle of readdir, the readdir
result may contain duplicated entries. (but there shouldn't be
any missing entries)

Fixes: ceph#14377
Signed-off-by: Yan, Zheng <zyan@redhat.com>
@ukernel ukernel added bug-fix cephfs Ceph File System labels Jan 15, 2016
@gregsfortytwo
Copy link
Member

@ukernel, please document what testing you performed to make sure this is good. (I assume you ran it against a vstart cluster or something? It looks good but don't want to pull it into an integration branch and have it break things because of a misplaced loop or something.)

gregsfortytwo added a commit that referenced this pull request Jan 18, 2016
…-fs-testing

#7246

Reviewed-by: Greg Farnum <gfarnum@redhat.com>
gregsfortytwo added a commit that referenced this pull request Jan 19, 2016
…-fs-testing

#7246

Reviewed-by: Greg Farnum <gfarnum@redhat.com>
@gregsfortytwo
Copy link
Member

gregsfortytwo added a commit that referenced this pull request Jan 20, 2016
ceph_test_libcephfs: tolerate duplicated entries in readdir

Reviewed-by: Greg Farnum <gfarnum@redhat.com>
@gregsfortytwo gregsfortytwo merged commit e7107f8 into ceph:jewel Jan 20, 2016
@ukernel ukernel deleted the jewel-14377 branch March 10, 2016 07:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants