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

osd/: don't leak context for Blessed*Context or RecoveryQueueAsync #13342

Merged
merged 1 commit into from Feb 10, 2017

Conversation

Projects
None yet
3 participants
@athanatos
Contributor

athanatos commented Feb 9, 2017

This has always been a bug, but until
68defc2, nothing deleted those contexts
without calling complete().

Fixes: http://tracker.ceph.com/issues/18809
Bug shadowed until: 68defc2
Signed-off-by: Samuel Just sjust@redhat.com

osd/: don't leak context for Blessed*Context or RecoveryQueueAsync
This has always been a bug, but until
68defc2, nothing deleted those contexts
without calling complete().

Fixes: http://tracker.ceph.com/issues/18809
Bug shadowed until: 68defc2
Signed-off-by: Samuel Just <sjust@redhat.com>

@athanatos athanatos requested a review from jdurgin Feb 9, 2017

@athanatos

This comment has been minimized.

Contributor

athanatos commented Feb 9, 2017

@tchaikov tchaikov added the needs-qa label Feb 10, 2017

@athanatos athanatos merged commit 534ae8f into ceph:master Feb 10, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment