Bug 1298489

Summary: Unable to update VIRTIO_SCSI flag via rest api
Product: [oVirt] ovirt-engine Reporter: Tareq Alayan <talayan>
Component: RestAPIAssignee: Daniel Erez <derez>
Status: CLOSED CURRENTRELEASE QA Contact: Raz Tamir <ratamir>
Severity: high Docs Contact:
Priority: unspecified    
Version: 3.6.2.5CC: acanan, amureini, bugs, derez, juan.hernandez, ncredi, sbonazzo, talayan, tnisan, ylavi
Target Milestone: ovirt-3.6.2Keywords: Automation
Target Release: 3.6.2.6Flags: rule-engine: ovirt-3.6.z+
ylavi: planning_ack+
amureini: devel_ack+
rule-engine: testing_ack+
Hardware: x86_64   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-02-18 11:14:21 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:

Description Tareq Alayan 2016-01-14 09:20:27 UTC
Description of problem:
Unable to update VIRTIO_SCSI flag via rest api

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

How reproducible:
always

Steps to Reproduce:
1. Create a template with resource allocation:virtio_scsi is disabled (False)
2. update via rest  
PUT https://<engine-url>/api/templates/<template-id>
HEADER: Content-Type: application/xml
<template>
    <virtio_scsi enabled="true"/>
</template>

Actual results:
response body:
<virtio_scsi enabled="false"/>



Expected results:
<virtio_scsi enabled="true"/>


Additional info:
In short in rest api doesn't update the virtio_scci flag. 
Note that in the UI it is update it successfully.

Comment 1 Juan Hernández 2016-01-14 10:53:25 UTC
This happens with templates and also with instance types. The proposed patch fixes the issue, and it has been verified it in the master and 3.6 branches.

Allon, can you assign the bug to someone in your team, so that she/he follows the bug to completion? Backporting to the 3.6 branch should be easy, just a cherry-pick, and I already verified it works.

Comment 2 Allon Mureinik 2016-01-14 11:57:23 UTC
(In reply to Juan Hernández from comment #1)
> This happens with templates and also with instance types. The proposed patch
> fixes the issue, and it has been verified it in the master and 3.6 branches.
> 
> Allon, can you assign the bug to someone in your team, so that she/he
> follows the bug to completion? Backporting to the 3.6 branch should be easy,
> just a cherry-pick, and I already verified it works.
Thanks Juan!
Daniel - please take this one over.

Comment 3 Daniel Erez 2016-01-14 13:38:19 UTC
(In reply to Allon Mureinik from comment #2)
> (In reply to Juan Hernández from comment #1)
> > This happens with templates and also with instance types. The proposed patch
> > fixes the issue, and it has been verified it in the master and 3.6 branches.
> > 
> > Allon, can you assign the bug to someone in your team, so that she/he
> > follows the bug to completion? Backporting to the 3.6 branch should be easy,
> > just a cherry-pick, and I already verified it works.
> Thanks Juan!
> Daniel - please take this one over.

Not sure how relevant for 3.6 since disks can't be added to an existing template any way. I.e. it could be needed only when creating a VM from a template, but then, can be changed from the new VM dialog.

Comment 4 Juan Hernández 2016-01-14 13:55:31 UTC
Tareq, Nelly, how important is this for automation? Needs to be fixed in 3.6.z?

Comment 5 Allon Mureinik 2016-01-14 13:59:38 UTC
(In reply to Juan Hernández from comment #4)
> Tareq, Nelly, how important is this for automation? Needs to be fixed in
> 3.6.z?

It's a trivial fix - let's just do it.

Comment 6 Tareq Alayan 2016-01-14 16:28:24 UTC
Yes it will be very nice to have

Comment 7 Raz Tamir 2016-01-21 11:49:15 UTC
Verified on rhevm-3.6.2.6-0.1.el6.noarch