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

kraken: rgw: multisite: some yields in RGWMetaSyncShardCR::full_sync() resume in incremental_sync() #13838

Merged
merged 1 commit into from Jun 20, 2017

Conversation

smithfarm
Copy link
Contributor

@smithfarm smithfarm self-assigned this Mar 7, 2017
@smithfarm smithfarm added this to the kraken milestone Mar 7, 2017
@smithfarm smithfarm changed the title kraken: multisite: some yields in RGWMetaSyncShardCR::full_sync() resume in incremental_sync() kraken: rgw: multisite: some yields in RGWMetaSyncShardCR::full_sync() resume in incremental_sync() Mar 7, 2017
@smithfarm
Copy link
Contributor Author

@yehudasa This passed an RGW suite at http://tracker.ceph.com/issues/19009#note-8

Please review/approve if you agree it can be merged.

@smithfarm smithfarm requested a review from yehudasa April 12, 2017 11:46
@smithfarm
Copy link
Contributor Author

@yehudasa This PR passed another RGW suite at http://tracker.ceph.com/issues/19009#note-18

Waiting for review.

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

Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit e62d48a)
@smithfarm
Copy link
Contributor Author

@yehudasa @cbodley @oritwas This has passed integration testing. OK to merge?

@oritwas
Copy link
Member

oritwas commented Jun 20, 2017

@smithfarm , you can merge it

@oritwas
Copy link
Member

oritwas commented Jun 20, 2017

jenkins test this please

@smithfarm
Copy link
Contributor Author

Jenkins re-test this please

@smithfarm smithfarm merged commit 4b1e8bc into ceph:kraken Jun 20, 2017
@smithfarm smithfarm deleted the wip-19049-kraken branch June 20, 2017 20:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants