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
mds: setup readonly mode for PurgeQueue #25621
Conversation
batrick
commented
Dec 19, 2018
b94b504
to
3879aaa
Compare
src/mds/PurgeQueue.cc
Outdated
dout(10) << "cannot create: PurgeQueue is readonly" << dendl; | ||
if (fin) | ||
fin->complete(-EROFS); | ||
return; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this does not make sense for creating new prugequeue. Instead, we should check error in finish context of writing journal head. (at later this function)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks, please have another look
Otherwise it leaks. Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
These could race with the asynchronous workings of the PQ. Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
This is a cheap refactor. Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
If the PQ faces an error, it should go read-only along with the MDS rank. Fixes: http://tracker.ceph.com/issues/37543 Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
* refs/pull/25621/head: mds: allow boot on read-only mds: setup readonly mode for PurgeQueue mds: return string_view for type str mds: add missing locks for PurgeQueue methods mds: delete on_error context on des Reviewed-by: Zheng Yan <zyan@redhat.com>