Change in vdsm[master]: sd: [wip] add getClusterLockOwnerId method

abaron at redhat.com abaron at redhat.com
Wed Nov 20 07:40:26 UTC 2013


Ayal Baron has posted comments on this change.

Change subject: sd: [wip] add getClusterLockOwnerId method
......................................................................


Patch Set 1:

(2 comments)

....................................................
Commit Message
Line 3: AuthorDate: 2013-11-19 10:25:16 -0500
Line 4: Commit:     Federico Simoncelli <fsimonce at redhat.com>
Line 5: CommitDate: 2013-11-19 10:40:26 -0500
Line 6: 
Line 7: sd: [wip] add getClusterLockOwnerId method
why?
Line 8: 
Line 9: Change-Id: I10e64d74319ea6591a7edf8e17809d367a758386


....................................................
File vdsm/storage/clusterlock.py
Line 263:         if len(owners) == 1:
Line 264:             return owners[0].get("host_id")
Line 265:         elif len(owners) > 1:
Line 266:             self.log.error("Cluster lock is reported to have more than "
Line 267:                            "one owner: %s", owners)
? how is it that sanlock would permit this?
Line 268:             raise RuntimeError("Cluster lock multiple owners error")
Line 269: 
Line 270:         return None
Line 271: 


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I10e64d74319ea6591a7edf8e17809d367a758386
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list