Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1346918

Summary: Remove old API of VM disks
Product: [oVirt] ovirt-engine Reporter: Tal Nisan <tnisan>
Component: RestAPIAssignee: Tal Nisan <tnisan>
Status: CLOSED CURRENTRELEASE QA Contact: Raz Tamir <ratamir>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 4.0.0CC: acanan, amureini, bugs, sbonazzo, tnisan, ylavi
Target Milestone: ovirt-4.0.1Keywords: CodeChange, ZStream
Target Release: 4.0.1.1Flags: rule-engine: ovirt-4.0.z+
rule-engine: blocker+
rule-engine: planning_ack+
tnisan: devel_ack+
rule-engine: testing_ack+
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-07-19 06:23:11 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Storage RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Tal Nisan 2016-06-15 16:04:12 UTC
Description of problem:

Currently V4 API still supports V3 lists and actions on VM disks in:

api/vms/{vm_id}/disks
api/vms/{vm_id}/disks/{disk_id}

Those should be removed and the functionality should only be supported in:

api/vms/{vm_id}/diskattachments
api/vms/{vm_id}/diskattachments/{disk_attachment_id}


The functionality in V4 resources used by delegation from V3 should be moved to V3 classes

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 Juan Hernández 2016-06-15 16:07:11 UTC
This kind of change can't be targeted to 4.0.1, as we are not allowed to make backwards compatibility breaking changes once version 4 of the API is released. Please retarget to 4.0.0, or close as WONTFIX.

Comment 2 Yaniv Lavi 2016-06-27 11:43:18 UTC
We will have a release note on removal of this. We should allow small changes in the first z streams of the 4.0 API.

Comment 3 Yaniv Lavi 2016-06-29 11:47:07 UTC
We don't have many users that already moved to v4 of the API therefore we can allow small breakages of backwards compatibility on a case by case request.
This one is approved.

Comment 4 Allon Mureinik 2016-07-07 12:11:11 UTC
Tal - this should be documented. Not sure if here or as part of a broader API documentation.

Comment 5 Raz Tamir 2016-07-17 18:08:04 UTC
Verified on rhevm-4.0.1.1-0.1.el7ev.noarch.
There is no link to api/vms/{vm_id}/disks with v4 api

Comment 6 Sandro Bonazzola 2016-07-19 06:23:11 UTC
Since the problem described in this bug report should be
resolved in oVirt 4.0.1 released on July 19th 2016, it has been closed with a
resolution of CURRENT RELEASE.

For information on the release, and how to update to this release, follow the link below.

If the solution does not work for you, open a new bug report.

http://www.ovirt.org/release/4.0.1/