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

mgr: fix session leak #14720

Merged
merged 3 commits into from Apr 21, 2017

Conversation

Projects
None yet
1 participant
@liewegas
Member

liewegas commented Apr 21, 2017

No description provided.

liewegas added some commits Apr 12, 2017

vstart.sh: add --valgrind_mgr
Signed-off-by: Sage Weil <sage@redhat.com>
mgr/MgrSession: allow ref tracing, and fix ref leak
The first RefCountedObj ctor arg is a cct, not a ref count.

Fixes: http://tracker.ceph.com/issues/19591
Signed-off-by: Sage Weil <sage@redhat.com>
vstart.sh: add --valgrind_args
To do leak checking, you can do

 vstart.sh ... \
	--valgrind_mgr memcheck \
	--valgrind_args '--leak-check=full --show-reachable=yes'

Signed-off-by: Sage Weil <sage@redhat.com>

@liewegas liewegas merged commit 97ebb08 into ceph:master Apr 21, 2017

2 of 3 checks passed

default Build triggered. sha1 is merged.
Details
Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details

@liewegas liewegas added the mgr label Apr 21, 2017

@liewegas

This comment has been minimized.

Member

liewegas commented Apr 21, 2017

merging these without #14092

@liewegas liewegas deleted the liewegas:wip-19591 branch Apr 21, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment