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

dnarayan at redhat.com dnarayan at redhat.com
Fri Sep 26 11:58:00 UTC 2014


Darshan N has posted comments on this change.

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


Patch Set 3:

(4 comments)

http://gerrit.ovirt.org/#/c/33312/3/vdsm/gluster/gfapi.py
File vdsm/gluster/gfapi.py:

Line 26: 
Line 27: import exception as ge
Line 28: from vdsm import constants
Line 29: from vdsm import utils
Line 30: from __init__ import makePublic
> Please move extra imports before workaround function
Done
Line 31: 
Line 32: 
Line 33: GLUSTER_VOL_PROTOCAL = 'tcp'
Line 34: GLUSTER_VOL_HOST = 'localhost'


Line 147: # This can be reverted back once the memory leak issue
Line 148: # is fixed in libgfapi.
Line 149: @makePublic
Line 150: def volumeStatvfs(volumeName):
Line 151:     script = constants.P_VDSM_GLUSTER + "gfapi.pyc"
> instead of script have:
Done
Line 152:     command = [constants.EXT_PYTHON, script, "-v", volumeName]
Line 153:     rc, out, err = utils.execCmd(command)
Line 154:     if rc != 0:
Line 155:         raise ge.GlfsStatvfsException(rc, out, err)


Line 148: # is fixed in libgfapi.
Line 149: @makePublic
Line 150: def volumeStatvfs(volumeName):
Line 151:     script = constants.P_VDSM_GLUSTER + "gfapi.pyc"
Line 152:     command = [constants.EXT_PYTHON, script, "-v", volumeName]
> Here:
Done
Line 153:     rc, out, err = utils.execCmd(command)
Line 154:     if rc != 0:
Line 155:         raise ge.GlfsStatvfsException(rc, out, err)
Line 156:     json_acceptable_string = out[0].replace("'", "\"")


Line 183:     args = parser.parse_args()
Line 184:     return args
Line 185: 
Line 186: 
Line 187: if __name__ == '__main__':
> This will get executed by the python interpreter when putting -m and pointi
Done
Line 188:     args = parse_cmdargs()
Line 189:     try:
Line 190:         res = volumeStatvfsGet(args.volume, args.host,
Line 191:                                args.port, args.protocol)


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7079426178ce47008d9a3b83635afce98536ca34
Gerrit-PatchSet: 3
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
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list