Bug 1337257

Summary:	[scale][ovirt-3.6.z] - GetStorageConnectionsByStorageTypeAndStatus generate slow query for multiple SD's
Product:	[oVirt] ovirt-engine	Reporter:	Allon Mureinik <amureini>
Component:	Database.Core	Assignee:	Allon Mureinik <amureini>
Status:	CLOSED CURRENTRELEASE	QA Contact:	guy chen <guchen>
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	3.6.2	CC:	acanan, amureini, bugs, emarcian, gklein, mperina, oourfali, stirabos, ylavi
Target Milestone:	ovirt-3.6.7	Flags:	amureini: ovirt-3.6.z? rule-engine: planning_ack? amureini: devel_ack+ rule-engine: testing_ack+
Target Release:	3.6.7.3
Hardware:	x86_64
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:	1306325	Environment:
Last Closed:	2016-07-04 12:31:54 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:
Bug Depends On:	1306325
Bug Blocks:

Description Allon Mureinik 2016-05-18 15:54:39 UTC

+++ This bug was initially created as a clone of Bug #1306325 +++

Description of problem:
SP GetStorageConnectionsByStorageTypeAndStatus response time grow when running 
many SD's

--10 rows \ SD's exe time = ~3.2sec

--1 row \ SD exe time = ~1.2sec

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

How reproducible:
100%

Steps to Reproduce:
1. 10K vms 12 SD's

Actual results:
execution time ~3 sec and above.

Expected results:
less than 1 sec.

Additional info:

--- Additional comment from Eldad Marciano on 2016-02-10 17:20:55 IST ---

performance improved when using this patch:
https://gerrit.ovirt.org/#/c/52912/

depends on:
https://bugzilla.redhat.com/show_bug.cgi?id=1302794

--- Additional comment from Eldad Marciano on 2016-02-10 17:21:20 IST ---



--- Additional comment from Eldad Marciano on 2016-02-10 17:22:40 IST ---

--10 rows \ SD's exe time = 485ms
--1 row \ SD exe time = 495ms

--- Additional comment from Eli Mesika on 2016-03-29 16:43:17 IDT ---

So, IIUC from comment 1 https://gerrit.ovirt.org/#/c/52912/ solved the issue 
Please approve

--- Additional comment from Sandro Bonazzola on 2016-05-02 12:51:27 IDT ---

Moving from 4.0 alpha to 4.0 beta since 4.0 alpha has been already released and bug is not ON_QA.

--- Additional comment from Eldad Marciano on 2016-05-03 14:31:14 IDT ---

approve.
any chance to back-port?

--- Additional comment from Martin Perina on 2016-05-03 14:38:09 IDT ---

Allon, do you agree backporting https://gerrit.ovirt.org/52912 into 3.6?

--- Additional comment from Allon Mureinik on 2016-05-05 15:31:10 IDT ---

(In reply to Martin Perina from comment #7)
> Allon, do you agree backporting https://gerrit.ovirt.org/52912 into 3.6?

The backport is a bit tricky, but it's easy to verify, and once verified it shouldn't be too risky. 
Taking the BZ, and will handle the BZ for 3.6.7, as to not add additional risk to 3.6.6

--- Additional comment from Allon Mureinik on 2016-05-18 18:53:16 IDT ---

Messed up the flags a bit here, sorry. 
This bug is ON_QA for 4.0 alpha release.
I'll clone it to a new one for 3.6.7 since the schemas have diverged quite a bit, and I'd like it to be verified on both branches.
------------------------------------------------------------------------------
This BZ is the aforementioned clone for the 3.6.7 branch, so QA can verify them both idenpendently.

Comment 1 Red Hat Bugzilla Rules Engine 2016-05-18 15:54:47 UTC

Bug tickets must have version flags set prior to targeting them to a release. Please ask maintainer to set the correct version flags and only then set the target milestone.