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

mds: propagate error encountered during opening inode by number #12749

Merged
merged 2 commits into from Jan 13, 2017

Conversation

Projects
None yet
2 participants
@ukernel
Member

ukernel commented Jan 3, 2017

Fixes: http://tracker.ceph.com/issues/18179
Signed-off-by: Yan, Zheng zyan@redhat.com

@jcsp

This comment has been minimized.

Contributor

jcsp commented Jan 9, 2017

This is quite subtle, so it could do with a test (I'm having a hard time being completely sure that the EIO really gets returned in the end). I think that the new TestDamage.test_corrupt_backtrace that I just added in master is almost the same case. You could just copy that test and instead of corrupting the backtrace, remove the dirfrag object for the inode that the hardlink points to.

@jcsp jcsp assigned jcsp and ukernel and unassigned jcsp Jan 9, 2017

ukernel added some commits Jan 3, 2017

qa/tasks: add test_open_ino_errors
Validate that errors encountered during opening inos are properly
propagated

Signed-off-by: Yan, Zheng <zyan@redhat.com>
@ukernel

This comment has been minimized.

@jcsp

jcsp approved these changes Jan 13, 2017

@jcsp jcsp merged commit 2076cda into ceph:master Jan 13, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details

@ukernel ukernel deleted the ukernel:wip-18179 branch Feb 27, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment