Bug 1194291

Summary:	[RFE] REST API is calling a query _per VM_ in order to re-populate the display-related values -> potential engine overload
Product:	[oVirt] ovirt-engine	Reporter:	Einav Cohen <ecohen>
Component:	RFEs	Assignee:	bugs <bugs>
Status:	CLOSED DUPLICATE	QA Contact:	Pavel Stehlik <pstehlik>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	---	CC:	bugs, gklein, lsurette, rbalakri, s.kieske, srevivo, tjelinek, ykaul
Target Milestone:	---	Keywords:	FutureFeature, Improvement
Target Release:	---	Flags:	ylavi: ovirt-future? ylavi: planning_ack? ylavi: devel_ack? ylavi: testing_ack?
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Enhancement
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2016-12-06 09:57:28 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:
Embargoed:

Description Einav Cohen 2015-02-19 13:40:47 UTC

In patch http://gerrit.ovirt.org/#/c/35281, it seems that the REST API is calling a query *per VM* in order to re-populate the display-related values. 

This will potentially overload the engine, especially if we are planning to move the GUI to work over the REST API and request VMs every 5 seconds (by default) per client, as we are doing today. 

Improvement suggestion from Juan [1]:
"""
I'd suggest to implement at least a query that takes multiple VM identifiers and returns the graphics devices of those multiple VMs. This would allow for a 1+1 queries scenario, instead of 1+N. Once the backend provides this the RESTAPI can be changed to use it.
"""

[1] http://lists.ovirt.org/pipermail/devel/2015-February/009900.html

Comment 2 Tomas Jelinek 2016-12-06 09:57:28 UTC


*** This bug has been marked as a duplicate of bug 1268216 ***