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: Journal needs to handle duplicate maintenance op tids #10045

Merged
1 commit merged into from Aug 9, 2016
Merged

jewel: rbd: Journal needs to handle duplicate maintenance op tids #10045

1 commit merged into from Aug 9, 2016

Conversation

ghost
Copy link

@ghost ghost commented Jun 30, 2016

Fixes: http://tracker.ceph.com/issues/16362
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
(cherry picked from commit 3a5b491)
@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
…aintenance op tids

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Jul 1, 2016
…aintenance op tids

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
…aintenance op tids

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

trociny commented Aug 8, 2016

lgtm

@ghost ghost merged commit b86aec9 into ceph:jewel Aug 9, 2016
@ghost ghost changed the title jewel: Journal needs to handle duplicate maintenance op tids jewel: rbd: Journal needs to handle duplicate maintenance op tids 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