Bug 1376345

Summary: [REST] Remove VM: Disk is deleted although choose detach only
Product: [oVirt] ovirt-engine Reporter: Israel Pinto <ipinto>
Component: BLL.VirtAssignee: Michal Skrivanek <michal.skrivanek>
Status: CLOSED NOTABUG QA Contact: meital avital <mavital>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 4.0.2.1CC: bugs, tjelinek
Target Milestone: ---Flags: ipinto: planning_ack?
ipinto: devel_ack?
ipinto: testing_ack?
Target Release: ---   
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-09-16 07:02:02 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Virt RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
engine.log none

Description Israel Pinto 2016-09-15 08:09:00 UTC 
Created attachment 1201156 [details]
engine.log

Description of problem:
Remove VM with disk and choose not to remove disk via REST.

REST request:

DELETE /api/vms/<uuid> HTTP/1.1
Accept: application/xml
Content-type: application/xml

<action>
    <vm>
        <disks>
            <detach_only>true</detach_only>
        </disks>
    </vm>
</action>

also try with diskattachments: 
<action>
    <vm>
        <diskattachments>
            <detach_only>true</detach_only>
        </diskattachments>
    </vm>
</action>

Version-Release number of selected component (if applicable):
rhevm-4.0.4.2-0.1.el7ev.noarch

How reproducible:
100 %

Steps to Reproduce:
1. Create Vm with disk
2. Remove VM and de-attach disk with REST 

Actual results:
VM and Disk deleted

Expected results:
VM will be deleted 
Disk will not be deleted
Comment 1 Tomas Jelinek 2016-09-16 07:02:02 UTC 
The <detach_only>true</detach_only> has been replaced by url param. For details please check the commit message of: https://gerrit.ovirt.org/#/c/46734/