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: rgw: multisite: memory leak in RGWCloneMetaLogCoroutine::state_store_mdlog_entries() #13179

Closed
wants to merge 3 commits into from

Conversation

smithfarm
Copy link
Contributor

@smithfarm smithfarm commented Jan 29, 2017

Fixes: http://tracker.ceph.com/issues/18407
Fixes: http://tracker.ceph.com/issues/18414

Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit 7a4975a)
RGWAsyncRadosRequest does not need a reference to its calling coroutine,
it only needs the completion notifier. this eliminates a circular
reference between RGWAsyncRadosRequest and the coroutines that create
them

Fixes: http://tracker.ceph.com/issues/17792
Fixes: http://tracker.ceph.com/issues/17793

Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit 7f670c0)
Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit 1d586f7)
@smithfarm smithfarm self-assigned this Jan 29, 2017
@smithfarm smithfarm added this to the jewel milestone Jan 29, 2017
@smithfarm smithfarm changed the title jewel: multisite: memory leak in RGWCloneMetaLogCoroutine::state_store_mdlog_entries() jewel: rgw: multisite: memory leak in RGWCloneMetaLogCoroutine::state_store_mdlog_entries() Jan 29, 2017
@smithfarm smithfarm added rgw and removed core labels Feb 1, 2017
@smithfarm
Copy link
Contributor Author

Duplicate of #13001

@smithfarm smithfarm closed this Feb 1, 2017
@smithfarm smithfarm deleted the wip-18560-jewel branch February 1, 2017 22:37
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.

2 participants