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
rbd-mirror: replicate image metadata settings #11168
Conversation
This needs #10980 to be merged first to avoid merge conflicts. |
return r; | ||
} | ||
} | ||
|
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.
Minor: check for m_image_ctx.read_only?
op_event->on_op_finish_event = new C_RefreshIfRequired<I>( | ||
m_image_ctx, new ExecuteOp<I, journal::MetadataRemoveEvent>( | ||
m_image_ctx, event, on_op_complete)); | ||
|
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.
Minor: add "op_event->ignore_error_codes = {-ENOENT}" to ignore replay errors
Minor: can you add new test cases to test_mock_Replay and test_Replay for the new events? |
@dillaman Thanks for the comments. Updated. |
@trociny Still would like to see a journal/test_mock_Replay test case |
Signed-off-by: Mykola Golub <mgolub@mirantis.com>
Fixes: http://tracker.ceph.com/issues/16212 Signed-off-by: Mykola Golub <mgolub@mirantis.com>
Fixes: http://tracker.ceph.com/issues/16212
Signed-off-by: Mykola Golub mgolub@mirantis.com