Change in ovirt-engine[master]: restapi: Use matrix parameter to force remove virtual machine

juan.hernandez at redhat.com juan.hernandez at redhat.com
Mon Nov 2 07:56:32 UTC 2015


Juan Hernandez has submitted this change and it was merged.

Change subject: restapi: Use matrix parameter to force remove virtual machine
......................................................................


restapi: Use matrix parameter to force remove virtual machine

The operation that removes a virtual machine supports a "force"
parameter. In order to use it the "DELETE" method used to support an
optional action parameter:

  DELETE /vms/{vm:id}
  <action>
    <force>true</force>
  </action>

This patch replaces the action parameter with an optional matrix
parameter:

  DELETE /vms/{vm:id};force=true

Change-Id: I0a3c1e189d23e9171179135a3f3552380d9cfb34
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/java/org/ovirt/engine/api/resource/VmResource.java
M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmResource.java
M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendVmResourceTest.java
5 files changed, 64 insertions(+), 46 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I0a3c1e189d23e9171179135a3f3552380d9cfb34
Gerrit-PatchSet: 3
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: automation at ovirt.org



More information about the Engine-commits mailing list