Change in ovirt-engine[master]: restapi: <placement_policy> should accept <host> by name (#7...

mpastern at redhat.com mpastern at redhat.com
Sun Nov 4 06:21:44 UTC 2012


Michael Pasternak has submitted this change and it was merged.

Change subject: restapi: <placement_policy> should accept <host> by name (#757115)
......................................................................


restapi: <placement_policy> should accept <host> by name (#757115)

The addition of a VM by host name doesn't work as intended. The
VM is being added without a host id.

This patch fixes the issue by passing the computed host id from host
name to the VmStatic class, which is used by the backend to create
the VM.

Change-Id: I09c9ad52db15e18813632598aca701e2b18af3d6
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=757115
Signed-off-by: Ravi Nori <rnori at redhat.com>
---
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmsResource.java
1 file changed, 1 insertion(+), 2 deletions(-)

Approvals:
  Michael Pasternak: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/8819
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I09c9ad52db15e18813632598aca701e2b18af3d6
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ravi Nori <rnori at redhat.com>
Gerrit-Reviewer: Michael Pasternak <mpastern at redhat.com>
Gerrit-Reviewer: Ravi Nori <rnori at redhat.com>



More information about the Engine-commits mailing list