Change in ovirt-engine[master]: restapi: Remove the capability to pin a VM to a single host

juan.hernandez at redhat.com juan.hernandez at redhat.com
Tue Sep 15 08:19:34 UTC 2015


Juan Hernandez has submitted this change and it was merged.

Change subject: restapi: Remove the capability to pin a VM to a single host
......................................................................


restapi: Remove the capability to pin a VM to a single host

In version 3.6 the capability to pin a VM to multiple hosts was added to
the RESTAPI, using the "hosts" element:

  PUT /vms/{vm:id}
  <vm>
    <placement_policy>
      <hosts>
        <host id="{host:id}/">
        <host id="{host:id}/">
        ...
      </hosts>
    </placement_policy>
  </vm>

The existing capability to pin to a single host was preserved, for
backwards compatibility. This patch removes that capability. It is
still possible to pin to a single host specifying a list of hosts with
one element:

  PUT /vms/{vm:id}
  <vm>
    <placement_policy>
      <hosts>
        <host id="{host:id}/">
      </hosts>
    </placement_policy>
  </vm>

Change-Id: I4e5de031b232e60523ff05088c848a5dab5c185c
Bug-Url: https://bugzilla.redhat.com/1260732
Signed-off-by: Juan Hernandez <juan.hernandez at redhat.com>
---
M backend/manager/modules/restapi/README.md
M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml
M backend/manager/modules/restapi/interface/definition/src/main/schema/api.xsd
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmsResource.java
M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendVmResourceTest.java
M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendVmsResourceTest.java
M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java
7 files changed, 55 insertions(+), 35 deletions(-)

Approvals:
  Juan Hernandez: Verified; Looks good to me, approved
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/45813
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I4e5de031b232e60523ff05088c848a5dab5c185c
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list