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

cls_rbd: mirroring directory #7620

Closed
wants to merge 2 commits into from
Closed

cls_rbd: mirroring directory #7620

wants to merge 2 commits into from

Conversation

jdurgin
Copy link
Member

@jdurgin jdurgin commented Feb 12, 2016

Move pool mirroring settings to an 'rbd_mirroring' object, and add a
directory of images that are mirrored, and their remote counterparts.

@jdurgin
Copy link
Member Author

jdurgin commented Feb 12, 2016

Added some thoughts on what should go on top of this at the end of http://pad.ceph.com/p/rbd_mirror_daemon

@dillaman
Copy link

Minor: should the new type be added to the ceph-dencoder test?

@jdurgin
Copy link
Member Author

jdurgin commented Feb 17, 2016

yeah, meant to add that

We'll use this object only for mirroring-related purposes, not generic
settings on a pool.

Refs: ceph#14419
Signed-off-by: Josh Durgin <jdurgin@redhat.com>
These will track whether local images should be mirrored, and map them
to a unique global id. There's a state field for safely disabling
mirroring while operating on multiple objects.

Fixes: ceph#14419
Signed-off-by: Josh Durgin <jdurgin@redhat.com>
@dillaman dillaman self-assigned this Feb 24, 2016
dillaman pushed a commit that referenced this pull request Feb 24, 2016
cls_rbd: mirroring directory

Reviewed-by: Jason Dillaman <dillaman@redhat.com>
@dillaman
Copy link

Merged manually -- TestClsRbd.mirror_image test case would fail if TestClsRbd.mirror test case run beforehand.

@dillaman dillaman closed this Feb 24, 2016
@jdurgin jdurgin deleted the wip-14419 branch February 26, 2016 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants