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: tools: ceph-objectstore-tool crashes if --journal-path <a-directory> #11407

Merged
1 commit merged into from Oct 19, 2016
Merged

jewel: tools: ceph-objectstore-tool crashes if --journal-path <a-directory> #11407

1 commit merged into from Oct 19, 2016

Conversation

ghost
Copy link

@ghost ghost commented Oct 11, 2016

… or regular file

otherwise JournalingFileStore will assert when deleting FileJournal
which still has the non block/regular file opened.

Fixes: http://tracker.ceph.com/issues/17307
Signed-off-by: Kefu Chai <kchai@redhat.com>
(cherry picked from commit 7431eec)
@ghost ghost self-assigned this Oct 11, 2016
@ghost ghost added this to the jewel milestone Oct 11, 2016
@ghost ghost added bug-fix core labels Oct 11, 2016
ghost pushed a commit that referenced this pull request Oct 13, 2016
…journal-path <a-directory>

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

ghost commented Oct 14, 2016

@tchaikov does this backport look good to merge ? It passed the rados (http://tracker.ceph.com/issues/17487#note-12), upgrade/jewel-x and upgrade/hammer-x (http://tracker.ceph.com/issues/17487#note-15) suites (except for two bugs that are, I believe, unrelated)

@ghost ghost assigned tchaikov Oct 14, 2016
ghost pushed a commit that referenced this pull request Oct 17, 2016
…journal-path <a-directory>

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Oct 18, 2016
…journal-path <a-directory>

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@tchaikov tchaikov removed their assignment Oct 19, 2016
@tchaikov
Copy link
Contributor

lgtm.

@ghost ghost merged commit d15d6dc into ceph:jewel Oct 19, 2016
@theanalyst theanalyst changed the title jewel: ceph-objectstore-tool crashes if --journal-path <a-directory> "jewel: tools: ceph-objectstore-tool crashes if --journal-path <a-directory>" Nov 17, 2016
@theanalyst theanalyst changed the title "jewel: tools: ceph-objectstore-tool crashes if --journal-path <a-directory>" jewel: tools: ceph-objectstore-tool crashes if --journal-path <a-directory> Nov 17, 2016
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant