Change in ovirt-engine[master]: restapi: Build correctly creation status links

juan.hernandez at redhat.com juan.hernandez at redhat.com
Mon Sep 19 12:29:32 UTC 2016


Juan Hernandez has submitted this change and it was merged.

Change subject: restapi: Build correctly creation status links
......................................................................


restapi: Build correctly creation status links

The code that builds the links returned by the creation status resource
doesn't build correctly the links, because it doesn't take into account
the source of the API version. As a result the links contain always the
version prefix. For example:

  <creation href="/ovirt-engine/api/v4/vms/..." id="...">
    <status>complete</status>
  </creation>

Note the '/v4' suffix, it should be there only if it is included in the
request URI, but it is currently always present, even when the version
is selected using the 'Version' header.

This patch fixes the creation status resource so that it builds the link
taking into account where the version has been specified.

Change-Id: I9cc0f7f03d2dd5ece00ed0081a7fb058cd5b7a27
Related-To: https://bugzilla.redhat.com/1359139
Signed-off-by: Juan Hernandez <juan.hernandez at redhat.com>
---
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/AbstractBackendAsyncStatusResource.java
1 file changed, 12 insertions(+), 13 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I9cc0f7f03d2dd5ece00ed0081a7fb058cd5b7a27
Gerrit-PatchSet: 2
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: Ori Liel <oliel at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list