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@redhat.com
Gerrit-Reviewer: Adam Litke
alitke@redhat.com
Gerrit-Reviewer: Allon Mureinik
amureini@redhat.com
Gerrit-Reviewer: Dan Kenigsberg
danken@redhat.com
Gerrit-Reviewer: Daniel Erez
derez@redhat.com
Gerrit-Reviewer: Federico Simoncelli
fsimonce@redhat.com
Gerrit-Reviewer: Liron Aravot
laravot@redhat.com
Gerrit-Reviewer: Maor Lipchuk
mlipchuk@redhat.com
Gerrit-Reviewer: Martin Peřina
mperina@redhat.com
Gerrit-Reviewer: Nir Soffer
nsoffer@redhat.com
Gerrit-Reviewer: Saggi Mizrahi
smizrahi@redhat.com
Gerrit-Reviewer: automation@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No