Eli Mesika has uploaded a new change for review. Change subject: core:fix adding host from REST API ......................................................................
core:fix adding host from REST API Fixing a regression resulted from commit 31cd7918c9c5eb5c2d6ddf5526b366fdb2d0f4bc (PM refactoring) The regression caused a host added using cluster name to fail. Change-Id: I527e79102ae7af5d2bb6703e20dd388677da30c2 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1176391 Signed-off-by: emesika <[email protected]> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostsResource.java 1 file changed, 1 insertion(+), 3 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/17/36317/1 diff --git a/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostsResource.java b/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostsResource.java index a2af7dd..c0520db 100644 --- a/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostsResource.java +++ b/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostsResource.java @@ -95,9 +95,7 @@ validateParameters(host, "name", "address"); VdsStatic staticHost = getMapper(Host.class, VdsStatic.class).map(host, null); staticHost.setVdsGroupId(getClusterId(host)); - AddVdsActionParameters addParams = - new AddVdsActionParameters(getMapper(Host.class, VdsStatic.class).map(host, null), - host.getRootPassword()); + AddVdsActionParameters addParams = new AddVdsActionParameters(staticHost, host.getRootPassword()); if (host.isSetOverrideIptables()) { addParams.setOverrideFirewall(host.isOverrideIptables()); } -- To view, visit http://gerrit.ovirt.org/36317 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I527e79102ae7af5d2bb6703e20dd388677da30c2 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
