Change in vdsm[master]: gluster: Temporary fix for supervdsm memory leak.

danken at redhat.com danken at redhat.com
Mon Sep 29 11:23:06 UTC 2014


Dan Kenigsberg has submitted this change and it was merged.

Change subject: gluster: Temporary fix for supervdsm memory leak.
......................................................................


gluster: Temporary fix for supervdsm memory leak.

     Supervdsm memory leak is caused by memory leak in libgfapi
(BZ:1093594) which is used by verb glusterVolumeStatsInfoGet to
retrive the volume statistics. The libgfapi method glfs_fini()
is not cleaning up the memory, so everytime the verb is invoked
memory gets added up(~20-30MiB). This verb is invoked by engine
every 5 minutes for volume capacity feature. So supervdsm memory
usage shoots up when gluster is enabled.

    This patch provides a temporary fix until libgfapi issue is
resolved. In this patch libgfapi method calls are executed as
script using execCmd(), by this libgfapi invocation is not part
of supervdsm process.

Change-Id: I7079426178ce47008d9a3b83635afce98536ca34
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1142647
Signed-off-by: ndarshan <dnarayan at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/33312
Reviewed-by: Antoni Segura Puimedon <asegurap at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M vdsm/gluster/gfapi.py
1 file changed, 78 insertions(+), 3 deletions(-)

Approvals:
  Antoni Segura Puimedon: Looks good to me, but someone else must approve
  Darshan N: Verified
  Dan Kenigsberg: Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/33312
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I7079426178ce47008d9a3b83635afce98536ca34
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Darshan N <dnarayan at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Bala.FA <barumuga at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Darshan N <dnarayan at redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list