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: ignore ENOENT on writing backtrace #14207

Merged
merged 1 commit into from Apr 15, 2017

Conversation

Projects
None yet
2 participants
@jcsp
Contributor

jcsp commented Mar 28, 2017

We get ENOENT when a pool doesn't exist. This can
happen because we don't prevent people deleting
former cephfs data pools whose files may not have
had their metadata flushed yet.

http://tracker.ceph.com/issues/19401
Signed-off-by: John Spray john.spray@redhat.com

@ukernel

This comment has been minimized.

Member

ukernel commented Mar 29, 2017

-ENOENT is used for lots of error cases, a little worried

John Spray
mds: ignore ENOENT on writing backtrace
We get ENOENT when a pool doesn't exist.  This can
happen because we don't prevent people deleting
former cephfs data pools whose files may not have
had their metadata flushed yet.

http://tracker.ceph.com/issues/19401
Signed-off-by: John Spray <john.spray@redhat.com>
@jcsp

This comment has been minimized.

Contributor

jcsp commented Apr 13, 2017

Updated to only ignore ENOENT in the pool is really gone

@jcsp jcsp requested a review from ukernel Apr 13, 2017

@jcsp

This comment has been minimized.

Contributor

jcsp commented Apr 14, 2017

@jcsp jcsp merged commit f412345 into ceph:master Apr 15, 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

@jcsp jcsp deleted the jcsp:wip-19401 branch Apr 15, 2017

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