Change in ovirt-engine[ovirt-engine-4.0]: restapi: Build correctly creation status links
tnisan at redhat.com
tnisan at redhat.com
Mon Sep 19 13:01:37 UTC 2016
Tal Nisan 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>
(cherry picked from commit df4292de31b3a2ca3bd030da85e8e70086331c7c)
---
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/64132
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I9cc0f7f03d2dd5ece00ed0081a7fb058cd5b7a27
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.0
Gerrit-Owner: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
More information about the Engine-commits
mailing list