Change in vdsm[master]: Replace SPM domain locks with SANLock

fsimonce at redhat.com fsimonce at redhat.com
Fri Jan 6 09:02:20 UTC 2012


Federico Simoncelli has posted comments on this change.

Change subject: Replace SPM domain locks with SANLock
......................................................................


Patch Set 1: (1 inline comment)

....................................................
File vdsm/storage/sd.py
Line 99: # Lock Version
First safelease will disappear in the future (no need to have a domain version 3 with safelease), second If we want to provide a way to make it configurable (how?) then we don't need a new version at all. Probably if we believe it is possible, the domain version 2 will be the one with the configurable locking system. Third, domain version 3 at the moment is something hidden and we can change its format as we want (maybe adding what you want later on).

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3958a171e35d65544e0f2c3593daaf7daf8750ef
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Eduardo <ewarszaw at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>


More information about the vdsm-patches mailing list