Martin Peřina has posted comments on this change. Change subject: api: Add fencingPolicy parameter to fenceNode ......................................................................
Patch Set 1: 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/30762 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I7b1fd9521cebea28d0402e53d46d74b95e73f383 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Martin Peřina <[email protected]> Gerrit-Reviewer: Barak Azulay <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Martin Peřina <[email protected]> Gerrit-Reviewer: Oved Ourfali <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
