Navigation Menu

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: radosgw daemon core when reopen logs #10868

Merged
merged 1 commit into from Oct 13, 2016
Merged

jewel: rgw: radosgw daemon core when reopen logs #10868

merged 1 commit into from Oct 13, 2016

Conversation

ghost
Copy link

@ghost ghost commented Aug 25, 2016

the rgw_user_st_syn thread use the pointer meta_mgr in RGWUserStatsCache::sync_all_users() fuction, so we should close the thread before delete the pointer in RGWRados::finalize() function when close storage before reopen_logs.

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

Signed-off-by: weiqiaomiao <wei.qiaomiao@zte.com.cn>
(cherry picked from commit ea5d7fd)
@ghost ghost self-assigned this Aug 25, 2016
@ghost ghost added this to the jewel milestone Aug 25, 2016
@ghost ghost added bug-fix rgw labels Aug 25, 2016
ghost pushed a commit that referenced this pull request Aug 25, 2016
…n logs

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

ghost commented Aug 25, 2016

test this please (bluefs crash)

ghost pushed a commit that referenced this pull request Oct 10, 2016
…n logs

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Oct 10, 2016
…n logs

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Oct 13, 2016
…n logs

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost ghost assigned cbodley Oct 13, 2016
@ghost
Copy link
Author

ghost commented Oct 13, 2016

@cbodley does this backport look good to merge ? It passed the rgw suite http://tracker.ceph.com/issues/17487#note-11.

@cbodley cbodley merged commit e3839ab into ceph:jewel Oct 13, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants