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