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 <[email protected]>
Signed-off-by: Martin Perina <[email protected]>
---
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 <[email protected]>
Gerrit-Reviewer: Martin Peřina <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
