Yoav Kleinberger has posted comments on this change. Change subject: fencing: Introduce getHostLeaseStatus API ......................................................................
Patch Set 5: (2 comments) http://gerrit.ovirt.org/#/c/28873/5/vdsm/storage/clusterlock.py File vdsm/storage/clusterlock.py: Line 41: SDM_LEASE_NAME = 'SDM' Line 42: SDM_LEASE_OFFSET = 512 * 2048 Line 43: Line 44: # Host status codes Line 45: HOST_UNKNOWN = sanlock.HOST_UNKNOWN > This is indeed nicer for the log and for the engine, but for code inside vd I'm OK with your suggestion regarding this issue. Line 46: HOST_FREE = sanlock.HOST_FREE Line 47: HOST_LIVE = sanlock.HOST_LIVE Line 48: HOST_FAIL = sanlock.HOST_FAIL Line 49: HOST_DEAD = sanlock.HOST_DEAD Line 56: class HostStatusNotAvailable(Exception): Line 57: """ Line 58: Raised when host status is not available because the cluster lock does not Line 59: support this operation, or raised an exception. Line 60: """ > Users of this class should not depend on sanlock, unlike os module. Think a if this future arrives, we can deal with it then. YAGNI Line 61: Line 62: Line 63: class SafeLease(object): Line 64: log = logging.getLogger("Storage.SafeLease") -- To view, visit http://gerrit.ovirt.org/28873 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iccd62e58a194aa0ceb0f5e2503b8ec7e4349971b Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Barak Azulay <bazu...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com> Gerrit-Reviewer: Itamar Heim <ih...@redhat.com> Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczew...@gmail.com> Gerrit-Reviewer: Saggi Mizrahi <smizr...@redhat.com> Gerrit-Reviewer: Xavi Francisco <xfran...@redhat.com> Gerrit-Reviewer: Yoav Kleinberger <yklei...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches