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

jewel: mds: enable start when session ino info is corrupt #14700

Merged
merged 2 commits into from
Jun 14, 2017

Conversation

smithfarm
Copy link
Contributor

John Spray added 2 commits April 20, 2017 23:50
Use this to get a nice human readable name
when available (also including the session id in
parentheses)

Signed-off-by: John Spray <john.spray@redhat.com>
(cherry picked from commit 0f89787)
Mitigates http://tracker.ceph.com/issues/16842

Signed-off-by: John Spray <john.spray@redhat.com>
(cherry picked from commit c39aaf9)

Conflicts:
	src/mds/InoTable.cc - no 5259683 in jewel
	src/mds/InoTable.h - no 5259683 in jewel
@smithfarm smithfarm self-assigned this Apr 20, 2017
@smithfarm smithfarm added this to the jewel milestone Apr 20, 2017
@smithfarm smithfarm changed the title jewel: Enable MDS to start when session ino info is corrupt jewel: mds: enable start when session ino info is corrupt Apr 20, 2017
@smithfarm smithfarm added cephfs Ceph File System and removed core labels Apr 20, 2017
@smithfarm smithfarm requested a review from jcsp April 27, 2017 18:47
@smithfarm
Copy link
Contributor Author

@jcsp This passed a rados run at http://tracker.ceph.com/issues/19538#note-49

It also went through an fs run with the following result: http://tracker.ceph.com/issues/19538#note-57 (I ruled that a "pass" - in my own mind - since the only reproducible failure is in the libcephfs-java workunit)

asheplyakov pushed a commit to asheplyakov/ceph that referenced this pull request May 18, 2017
…ino info is corrupt

Reviewed-by: Nathan Cutler <ncutler@suse.com>
@jcsp jcsp merged commit 27c915f into ceph:jewel Jun 14, 2017
@smithfarm smithfarm deleted the wip-19709-jewel branch June 14, 2017 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug-fix cephfs Ceph File System
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants