Change in vdsm[master]: task: streamlining lock acquiring api

Federico Simoncelli fsimonce at redhat.com
Mon Aug 18 11:48:42 UTC 2014


Federico Simoncelli has posted comments on this change.

Change subject: task: streamlining lock acquiring api
......................................................................


Patch Set 3:

(1 comment)

http://gerrit.ovirt.org/#/c/29729/3/vdsm/storage/task.py
File vdsm/storage/task.py:

Line 1325:             i += 1
Line 1326:         return s
Line 1327: 
Line 1328:     @misc.logskip("ResourceManager")
Line 1329:     def getExclusiveLock(self, namespace, *resources, **options):
> To make it more clear, we can rename "*resources" to "*names".
Have an utility function/method, something like getNestedResourceName(...)
Line 1330:         self._acquireResource(resourceManager.LockType.exclusive, namespace,
Line 1331:                               resources, options)
Line 1332: 
Line 1333:     @misc.logskip("ResourceManager")


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iac7eaa5ad577279a602651ab2df9e53ee58ac9d5
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Yoav Kleinberger <ykleinbe 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