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

Reply via email to