Change in vdsm[master]: hsm: deleteImage - lock image by imgUUID and sdUUID
Federico Simoncelli
fsimonce at redhat.com
Tue Jun 17 14:36:46 UTC 2014
Federico Simoncelli has posted comments on this change.
Change subject: hsm: deleteImage - lock image by imgUUID and sdUUID
......................................................................
Patch Set 1:
(1 comment)
http://gerrit.ovirt.org/#/c/28264/1/vdsm/storage/hsm.py
File vdsm/storage/hsm.py:
Line 1465: # vars.task.setDefaultException(se.ChangeMeError("%s" % args))
Line 1466: pool = self.getPool(spUUID)
Line 1467: dom = sdCache.produce(sdUUID=sdUUID)
Line 1468:
Line 1469: vars.task.getExclusiveLock(STORAGE, "%s_%s", imgUUID, sdUUID)
> from initial look seems that we need to smilar lock with the domain namespa
Let's use the same format:
(sduuid)_imageNS.(imguuid)
(please use the IMAGE_NAMESPACE constant)
Line 1470: vars.task.getSharedLock(STORAGE, sdUUID)
Line 1471: allVols = dom.getAllVolumes()
Line 1472: volsByImg = sd.getVolsOfImage(allVols, imgUUID)
Line 1473: if not volsByImg:
--
To view, visit http://gerrit.ovirt.org/28264
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I057b1f680276b1eb8a5d910260b7db4c4236fbf5
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Liron Ar <laravot 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