Hello Martin Peřina, I'd like you to do a code review. Please visit
http://gerrit.ovirt.org/33138 to review the following change. Change subject: core: wrong param name for fencing policy ...................................................................... core: wrong param name for fencing policy In the api schema and API.py the fencing policy param is named policy whereas engine passes fencingPolicy so value is not properly mapped. This patch fixes the naming. Change-Id: I291eb7973afa023e2f3d49bd09b966ce6ae9e333 Bug-Url: https://bugzilla.redhat.com/1143860 Signed-off-by: pkliczewski <piotr.kliczew...@gmail.com> Signed-off-by: Martin Perina <mper...@redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/JsonRpcVdsServer.java 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/38/33138/1 diff --git a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/JsonRpcVdsServer.java b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/JsonRpcVdsServer.java index 2507e7f..2cb6506 100644 --- a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/JsonRpcVdsServer.java +++ b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/JsonRpcVdsServer.java @@ -476,7 +476,7 @@ .withParameter("action", action) .withOptionalParameter("secure", secured) .withOptionalParameter("options", options) - .withOptionalParameterAsMap("fencingPolicy", fencingPolicy) + .withOptionalParameterAsMap("policy", fencingPolicy) .build(); Map<String, Object> response = new FutureMap(this.client, request).withResponseKey("power").withResponseType(String.class); -- To view, visit http://gerrit.ovirt.org/33138 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I291eb7973afa023e2f3d49bd09b966ce6ae9e333 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Piotr Kliczewski <piotr.kliczew...@gmail.com> Gerrit-Reviewer: Martin Peřina <mper...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches