Bug 1021186

Summary:	"Soundcard enabled" VM property is not exposed in REST API
Product:	Red Hat Enterprise Virtualization Manager	Reporter:	David Jaša <djasa>
Component:	ovirt-engine-restapi	Assignee:	Eldan Shachar <eshachar>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Pavel Novotny <pnovotny>
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	3.3.0	CC:	bazulay, eshachar, iheim, juan.hernandez, mavital, michal.skrivanek, ofrenkel, oramraz, rbalakri, Rhev-m-bugs, yeylon
Target Milestone:	---
Target Release:	3.5.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:	virt
Fixed In Version:	vt3	Doc Type:	Enhancement
Doc Text:	Feature: enable controlling sound card availability for vm with rest-api Reason: this was available only with UI, missing in rest-api Result: user can choose to enable/disable sound card for vm using rest-api	Story Points:	---
Clone Of:		Environment:
Last Closed:	2015-02-17 08:29:43 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	1142923, 1156165

Description David Jaša 2013-10-20 08:39:18 UTC

Description of problem:
"Soundcard Enabled" can only be set in the UI but it's not accessible via API.

Version-Release number of selected component (if applicable):
is19:
rhevm-restapi-3.3.0-0.26.master.el6ev.noarch
rhevm-backend-3.3.0-0.26.master.el6ev.noarc

How reproducible:
always

Steps to Reproduce:
1. perform HTTP GET https://rhevm.example.org/api/vms/VM_UUID
2.
3.

Actual results:
"soundcard enabled" equivalent is not available

Expected results:
"soundcard enabled" equivalent is displayed and can be modified via HTTP PUT

Additional info:

Comment 3 Juan Hernández 2014-02-25 15:10:25 UTC

Michal, looks like this won't make 3.4, can you retarget it?

Comment 4 Eyal Edri 2014-09-10 20:21:48 UTC

fixed in vt3, moving to on_qa.
if you believe this bug isn't released in vt3, please report to rhev-integ

Comment 5 Pavel Novotny 2014-09-12 11:36:16 UTC

rhevm-3.5.0-0.11.beta.el6ev.noarch (vt3).

Confirming that VM/pool VM and template entities have now attribute 'soundcard_enabled'. However it's visible in the response only with use of HTTP header 'All-Content=true'. But...

@Eldan: Is it really desired to have this general VM/template attribute hidden behind the All-Content header? Other similar attributes, like vm.delete_protected or vm.display.smartcard_enabled, are exposed without it.

Comment 6 Eldan Shachar 2014-09-14 07:18:43 UTC

Yes, we talked about it with our REST guy, the soundcard field requires an additional query to the backend\db because it's a device and not an integral part of the VM object like most VM REST fields. 

Because queries may affect performance the fields which require them are currently added under the 'All-Content' header (fields such as RNG and virtioSCSI).

Comment 7 Pavel Novotny 2014-09-15 10:05:56 UTC

(In reply to Eldan Shachar from comment #6)
> Yes, we talked about it with our REST guy, the soundcard field requires an
> additional query to the backend\db because it's a device and not an integral
> part of the VM object like most VM REST fields. 
> 
> Because queries may affect performance the fields which require them are
> currently added under the 'All-Content' header (fields such as RNG and
> virtioSCSI).

I understand, moving to VERIFIED then, rhevm-3.5.0-0.11.beta.el6ev.noarch (vt3).

Comment 8 Omer Frenkel 2015-02-17 08:29:43 UTC

RHEV-M 3.5.0 has been released