Hello Roy Golan, Omer Frenkel,
I'd like you to do a code review. Please visit
http://gerrit.ovirt.org/7145
to review the following change.
Change subject: BZ#847744 fenceNode API is missing the options argument
......................................................................
BZ#847744 fenceNode API is missing the options argument
Yes, ovirt-3.1 is happily released by now, but I find this bug so
devastating, that I would like us to consider a follow-up build of vdsm.
Change-Id: Ib2ce9b0f71040f9198413fa06c5d8768994842ec
Signed-off-by: Roy Golan <[email protected]>
Signed-off-by: Dan Kenigsberg <[email protected]>
Reviewed-on: http://gerrit.ovirt.org/7058
Reviewed-by: Dan Kenigsberg <[email protected]>
Reviewed-by: Omer Frenkel <[email protected]>
Tested-by: Omer Frenkel <[email protected]>
---
M vdsm/BindingXMLRPC.py
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/45/7145/1
diff --git a/vdsm/BindingXMLRPC.py b/vdsm/BindingXMLRPC.py
index 7c65e6a..e1a61f1 100644
--- a/vdsm/BindingXMLRPC.py
+++ b/vdsm/BindingXMLRPC.py
@@ -352,7 +352,7 @@
secure=False, options=''):
api = API.Global(self.cif)
return api.fenceNode(addr, port, agent, username, password,
- action, secure)
+ action, secure, options)
def setLogLevel(self, level):
api = API.Global(self.cif)
--
To view, visit http://gerrit.ovirt.org/7145
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib2ce9b0f71040f9198413fa06c5d8768994842ec
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.1
Gerrit-Owner: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Omer Frenkel <[email protected]>
Gerrit-Reviewer: Roy Golan <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
