jewel: fs: Log path as well as ino when detecting metadata damage #11418

Merged
merged 2 commits into from Oct 17, 2016

Projects

None yet

2 participants

jcsp added some commits Sep 6, 2016
@jcsp @ldachary jcsp mds: s/used_ions/used_inos/
Signed-off-by: John Spray <john.spray@redhat.com>
(cherry picked from commit bd3645f)
d52f190
@jcsp @ldachary jcsp mds: log path with CDir damage messages
Previously you just got the inode number, which
wasn't terribly useful for e.g. a missing fragment
object, as you couldn't readily resolve the parent
path.

Fixes: http://tracker.ceph.com/issues/16973
Signed-off-by: John Spray <john.spray@redhat.com>
(cherry picked from commit 439cd5e)

Conflicts:
	src/mds/CDir.cc: the go_bad() prototype which is part of the
        context of the patch has changed.
2c4e1c1
@dachary dachary self-assigned this Oct 11, 2016
@dachary dachary added this to the jewel milestone Oct 11, 2016
@jcsp jcsp was assigned by dachary Oct 14, 2016
@dachary
Member
dachary commented Oct 14, 2016

@jcsp does this backport look good to merge ? It passed the fs suite http://tracker.ceph.com/issues/17487#note-13 (except for one known bug that is, I believe, unrelated)

@jcsp
Contributor
jcsp commented Oct 17, 2016

👍

@dachary dachary merged commit 368c96c into ceph:jewel Oct 17, 2016

2 checks passed

Signed-off-by all commits in this PR are signed
Details
default Build finished.
Details
@theanalyst theanalyst changed the title from jewel: Log path as well as ino when detecting metadata damage to "jewel: fs: Log path as well as ino when detecting metadata damage" Nov 17, 2016
@theanalyst theanalyst changed the title from "jewel: fs: Log path as well as ino when detecting metadata damage" to jewel: fs: Log path as well as ino when detecting metadata damage Nov 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment