Note: This is a beta release of Red Hat Bugzilla 5.0. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Also email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback here.
Bug 1694612 - glusterd leaking memory when issued gluster vol status all tasks continuosly
Summary: glusterd leaking memory when issued gluster vol status all tasks continuosly
Alias: None
Product: GlusterFS
Classification: Community
Component: glusterd
Version: 5
Hardware: x86_64
OS: Linux
Target Milestone: ---
QA Contact:
Depends On: 1691164
Blocks: 1686255 1694610
TreeView+ depends on / blocked
Reported: 2019-04-01 09:05 UTC by Sanju
Modified: 2019-04-09 11:07 UTC (History)
9 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of: 1691164
Last Closed: 2019-04-09 11:07:35 UTC
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:

Attachments (Terms of Use)

System ID Priority Status Summary Last Updated Gerrit 22466 None Merged glusterd: fix txn-id mem leak 2019-04-09 11:07:34 UTC

Description Sanju 2019-04-01 09:05:13 UTC
Description of problem:
glusterd is leaking memory when issused "gluster vol status tasks" continuosly for 12 hours. The memory increase is from 250MB to 1.1GB. The increase have been 750 MB.

Version-Release number of selected component (if applicable):

How reproducible:

Steps to Reproduce:
1. On a six node cluster with brick-multiplexing enabled
2. Created 150 disperse volumes and 250 replica volumes and started them
3. Taken memory footprint from all the nodes
4. Issued "while true; do gluster volume status all tasks; sleep 2; done" with a time gap of 2 seconds 

Actual results:
Seen a memory increase of glusterd on Node N1 from 260MB to 1.1GB

Expected results:
glusterd memory shouldn't leak

Comment 1 Sanju 2019-04-01 09:07:54 UTC
Root cause:

There's a leak of a key setting in the dictionary priv->glusterd_txn_opinfo in every volume status all transaction when cli fetches the list of volume names as part of the first transaction.

Comment 2 Worker Ant 2019-04-01 09:12:52 UTC
REVIEW: (glusterd: fix txn-id mem leak) posted (#1) for review on release-5 by Sanju Rakonde

Comment 3 Worker Ant 2019-04-09 11:07:35 UTC
REVIEW: (glusterd: fix txn-id mem leak) merged (#2) on release-5 by Shyamsundar Ranganathan

Note You need to log in before you can comment on or make changes to this bug.