Martin Peřina has posted comments on this change.
Change subject: fencing: Make getHostLeaseStatus API public
......................................................................
Patch Set 2:
I wanted this additional parameter to cover all other possible parameters regarding to do
some logic in fenceNode method. That was the reason to name it fencing policy. And also
naming this fencingPolicy with Map type we won't need to modify API call in future, we
will just add more items into map).
About the map parameter needed for getHostLeaseStatus, you are right, I didn't look
into this new call too much. I will have to find out how will I get the data from engine
db, but anyway I will just add this map as another item in fencingPolicy map.
--
To view, visit
http://gerrit.ovirt.org/29157
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I415c1fee6256bf8d4e03ee542cc58e193162e9b8
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: Adam Litke <alitke(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com>
Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com>
Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No