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.

Sign up for GitHub

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

Jump to bottom

mds: MDS is crashed (mds/CDir.cc: 1391: FAILED assert(!is_complete())) #4886

Merged
merged 1 commit into from Jul 1, 2015
Merged

mds: MDS is crashed (mds/CDir.cc: 1391: FAILED assert(!is_complete())) #4886

merged 1 commit into from Jul 1, 2015

Conversation

theanalyst
Copy link
Member

@ukernel @theanalyst
mds: clear CDir::STATE_REJOINUNDEF after fetching dirfrag
d723e11 
Fixes: ceph#11541
Signed-off-by: Yan, Zheng <zyan@redhat.com>
(cherry picked from commit ab1e539)
@theanalyst theanalyst self-assigned this Jun 7, 2015
@theanalyst theanalyst added this to the hammer milestone Jun 7, 2015
@theanalyst theanalyst added bug-fix core cephfs Ceph File System and removed core labels Jun 7, 2015
@loic-bot
Copy link

loic-bot commented Jun 7, 2015

SUCCESS: http://jenkins.ceph.dachary.org/job/ceph/5726/

@gregsfortytwo
Copy link
Member

@theanalyst @dachary this has been hanging around for a while, is it blocked on something or just waiting in a queue? :)

@ghost
Copy link

ghost commented Jun 30, 2015

@gregsfortytwo it is waiting for @theanalyst to schedule a run, which could be anytime now.

@theanalyst
Copy link
Member Author

@dachary @gregsfortytwo I had scheduled a run a week back. There are no failures for fs. This patch is ready to merge

@theanalyst
Copy link
Member Author

BTW the current status is tracked at http://tracker.ceph.com/issues/11990

@ghost
Copy link

ghost commented Jun 30, 2015

@theanalyst I see that http://tracker.ceph.com/issues/11990#fs passed and contains this pull request. Since you were the one authoring the backport, the next step is to ask @gregsfortytwo if it is ready to merge. gregsfortytwo decision will be made easier because he can verify the suite passed. If gregsfortytwo decides it is good to merge he can either merge it himself or just write LGTM as a comment and you can merge it.

@theanalyst
Copy link
Member Author

@dachary thanks for the info. I'll update the status on all the current pull requests targeting hammer

@theanalyst
Copy link
Member Author

@gregsfortytwo this pull request has passed the integration tests targeted against hammer. Ready to merge?

@gregsfortytwo
Copy link
Member

LGTM.

theanalyst added a commit that referenced this pull request Jul 1, 2015
@theanalyst
Merge pull request #4886 from theanalyst/wip-11737-hammer
72ecd52 
Reviewed-by: Greg Farnum <gfarnum@redhat.com>
@theanalyst theanalyst merged commit 72ecd52 into ceph:hammer Jul 1, 2015
@ghost ghost changed the title MDS is crashed (mds/CDir.cc: 1391: FAILED assert(!is_complete())) mds: MDS is crashed (mds/CDir.cc: 1391: FAILED assert(!is_complete())) Aug 4, 2015
liewegas pushed a commit that referenced this pull request Dec 14, 2016
suites/fs: add cephfs_journal_tool test
0d37b8e 
Fixes: #4886

Signed-off-by: John Spray <john.spray@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@theanalyst theanalyst

Labels
bug-fix cephfs Ceph File System
Projects
None yet
Milestone
hammer
4 participants
@theanalyst @loic-bot @gregsfortytwo @ukernel