-
Notifications
You must be signed in to change notification settings - Fork 261
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
restapi: fix hrefs in single GET disksnapshot request #283
Conversation
2e241af
to
3841e2d
Compare
/ost |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this looked familiar - but the change I did involved a similar change related to vm snapshots (querying a specific one rather than going through all the vm snapshots) and not to disk snapshots
it looks correct, I'd separate out the settings of href to a different commit but as this is a small PR, that's fine
3841e2d
to
ddf4781
Compare
Rebased |
...i/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendDiskSnapshotResource.java
Show resolved
Hide resolved
ddf4781
to
35a28fa
Compare
Moving the query change to a separate patch, this PR now handles only the href fix |
35a28fa
to
34c53d3
Compare
34c53d3
to
e81394e
Compare
/ost |
e81394e
to
688e3ab
Compare
Fixing GET api/disks/{diskid}/disksnapshots/{snapshotid}: Correct href for <parent> and <disk> elements. 1. <disk> element was returned without href attribute 2. <parent> element had an incorrect href - buildParentHref() was used, although the element is not related to the parent entity resource, but rather represents the parent image of the snapshot. Bug-Url: https://bugzilla.redhat.com/2013697
688e3ab
to
0422d91
Compare
/ost |
Fixing GET api/disks/{diskid}/disksnapshots/{snapshotid}:
Correct href for and elements.
<disk>
element was returned without href attribute<parent>
element had an incorrect href - buildParentHref() wasused, although the element is not related to the parent entity
resource, but rather represents the parent image of the snapshot.