OpenContrail

Memory leak in Sandesh python library

Bug #1405245 reported by Édouard Thuleau
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Fix Committed
Medium
Sundaresan Rajangam
OpenContrail
Fix Committed
Undecided
Babu Shanmugam

Bug Description

During stress tests we found that the API daemon memory consumption continuously increasing.
I found we can stem it by simply disable the collector client of the Sandesh instance into the contrail API [2].

I was also able to reproduce the leak with the attached script on a devstack environment.

[1] https://github.com/Juniper/contrail-sandesh/blob/master/library/python/pysandesh/sandesh_base.py#L71
[2] https://github.com/Juniper/contrail-controller/blob/master/src/config/api-server/vnc_cfg_api_server.py#L357-L362

Tags: sandesh
Revision history for this message
Édouard Thuleau (ethuleau) wrote :
Raj Reddy (rajreddy)
Changed in juniperopenstack:
importance: Undecided → Medium
assignee: nobody → Sundaresan Rajangam (srajanga)
Revision history for this message
Babu Shanmugam (anbu-p) wrote :

Propese a patch https://review.opencontrail.org/#/c/5962/

Babu Shanmugam (anbu-p)
Changed in opencontrail:
assignee: nobody → Babu Shanmugam (anbu-p)
status: New → In Progress
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/5962
Committed: http://github.org/Juniper/contrail-sandesh/commit/f91ffb2e2cd3349e8940af7c5749ec3ea421ea6c
Submitter: Zuul
Branch: master

commit f91ffb2e2cd3349e8940af7c5749ec3ea421ea6c
Author: Babu Shanmugam <email address hidden>
Date: Tue Dec 30 09:49:32 2014 +0000

Avoid repeatedly allocating StreamHandler for SandeshLogger

Change-Id: I37dd478eaedd1dcb182f8819c426fe940dc4bb03
Closes-Bug: #1405245

Changed in juniperopenstack:
status: New → Fix Committed
Changed in opencontrail:
status: In Progress → Fix Committed
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/6076
Committed: http://github.org/Juniper/contrail-sandesh/commit/cc13118ccfb5bab06be65bb900f5307303c761fc
Submitter: Zuul
Branch: R2.0

commit cc13118ccfb5bab06be65bb900f5307303c761fc
Author: Babu Shanmugam <email address hidden>
Date: Tue Dec 30 09:49:32 2014 +0000

Avoid repeatedly allocating StreamHandler for SandeshLogger

Change-Id: I37dd478eaedd1dcb182f8819c426fe940dc4bb03
Closes-Bug: #1405245

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/6071
Committed: http://github.org/Juniper/contrail-sandesh/commit/ffd40b947d9e85b2d6d65698bcba2db0c51a02ef
Submitter: Zuul
Branch: R1.10

commit ffd40b947d9e85b2d6d65698bcba2db0c51a02ef
Author: Babu Shanmugam <email address hidden>
Date: Tue Dec 30 09:49:32 2014 +0000

Avoid repeatedly allocating StreamHandler for SandeshLogger

Note: Backported from master

Change-Id: I37dd478eaedd1dcb182f8819c426fe940dc4bb03
Closes-Bug: #1405245

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.