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: rbd: rbd-mirror: ensure replay status formatter has completed before stopping replay #10043

Merged
1 commit merged into from Aug 9, 2016
Merged

Conversation

ghost
Copy link

@ghost ghost commented Jun 30, 2016

Fixes: http://tracker.ceph.com/issues/16352
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
(cherry picked from commit cccdca4)
@ghost ghost self-assigned this Jun 30, 2016
@ghost ghost added this to the jewel milestone Jun 30, 2016
@ghost ghost added bug-fix rbd labels Jun 30, 2016
ghost pushed a commit that referenced this pull request Jun 30, 2016
…rmatter has completed before stopping replay

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Jul 1, 2016
…rmatter has completed before stopping replay

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

ghost commented Aug 8, 2016

@trociny does this backport look good to merge ? It passed the rbd suite http://tracker.ceph.com/issues/16344#note-11 (except for two failures that seem unrelated http://tracker.ceph.com/issues/16922 and http://tracker.ceph.com/issues/16921)

@ghost ghost assigned trociny Aug 8, 2016
ghost pushed a commit that referenced this pull request Aug 8, 2016
…rmatter has completed before stopping replay

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@trociny
Copy link
Contributor

trociny commented Aug 8, 2016

lgtm

@ghost ghost merged commit adcf5a7 into ceph:jewel Aug 9, 2016
@ghost ghost changed the title jewel: rbd-mirror: ensure replay status formatter has completed before stopping replay jewel: rbd: rbd-mirror: ensure replay status formatter has completed before stopping replay Aug 25, 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
Development

Successfully merging this pull request may close these issues.

1 participant