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: monitor assertion failure when deactivating mds in (invalid) fscid 0 #11862

Merged
4 commits merged into from Nov 14, 2016
Merged

jewel: monitor assertion failure when deactivating mds in (invalid) fscid 0 #11862

4 commits merged into from Nov 14, 2016

Conversation

ghost
Copy link

@ghost ghost commented Nov 9, 2016

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
(cherry picked from commit de6c36a)
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
(cherry picked from commit 8934e95)
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
(cherry picked from commit 59de991)
This allows us to reuse code in parse_filesystem and avoid
get_filesystem which may fail if the fscid does not exist. This would
result in the program (mon) aborting due to the uncaught exception.

Fixes: http://tracker.ceph.com/issues/17518

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
(cherry picked from commit edc78e4)
@ghost ghost self-assigned this Nov 9, 2016
@ghost ghost added this to the jewel milestone Nov 9, 2016
@ghost ghost added bug-fix cephfs Ceph File System labels Nov 9, 2016
@ghost ghost changed the base branch from jewel to jewel-next November 9, 2016 15:27
@ghost
Copy link
Author

ghost commented Nov 9, 2016

jenkins test this please (general jenkins failure)

ghost pushed a commit that referenced this pull request Nov 9, 2016
…tivating mds in (invalid) fscid 0

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost
Copy link
Author

ghost commented Nov 14, 2016

@jcsp does this backport look good to merge ? It passed the fs suite http://tracker.ceph.com/issues/17851#note-5 (except for valgrind failures and one java test failure that also fail on the jewel branch ). Note that it will not be included in 10.2.4, reason why it targets jewel-next.

@ghost ghost assigned jcsp Nov 14, 2016
@ghost ghost merged commit 1bc2a7e into ceph:jewel-next Nov 14, 2016
This pull request was closed.
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
2 participants