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: osd: leaked MOSDMap #14943

Merged
merged 1 commit into from Aug 23, 2017

Conversation

Projects
None yet
4 participants
@shinobu-x
Contributor

shinobu-x commented May 3, 2017

osd: fix occasional MOSDMap leak
_committed_osd_maps() may return early (without putting
the ref) on shutdown.

Fixes: http://tracker.ceph.com/issues/18293
Signed-off-by: Sage Weil <sage@redhat.com>
(cherry picked from commit a74632f)

@liewegas liewegas added this to the jewel milestone May 4, 2017

@liewegas liewegas added the bug fix label May 4, 2017

@smithfarm

This comment has been minimized.

Show comment
Hide comment
@smithfarm

smithfarm Jun 19, 2017

Contributor

Jenkins re-test this please

Contributor

smithfarm commented Jun 19, 2017

Jenkins re-test this please

@smithfarm smithfarm added the core label Jul 6, 2017

@smithfarm smithfarm requested a review from jdurgin Aug 23, 2017

@smithfarm

This comment has been minimized.

Show comment
Hide comment
@smithfarm

smithfarm Aug 23, 2017

Contributor

@liewegas @jdurgin This passed a rados suite with one failure ( a segfault - see http://tracker.ceph.com/issues/21063#note-2 ). Do you think it can be merged?

Contributor

smithfarm commented Aug 23, 2017

@liewegas @jdurgin This passed a rados suite with one failure ( a segfault - see http://tracker.ceph.com/issues/21063#note-2 ). Do you think it can be merged?

@liewegas

This comment has been minimized.

Show comment
Hide comment
@liewegas

liewegas Aug 23, 2017

Member
Member

liewegas commented Aug 23, 2017

@smithfarm smithfarm merged commit a70259a into ceph:jewel Aug 23, 2017

5 of 6 checks passed

Docs: build check Docs: failed with errors
Details
Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
Unmodified Submodules submodules for project are unmodified
Details
default Build finished.
Details
make check make check succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment