Bug 1337257 - [scale][ovirt-3.6.z] - GetStorageConnectionsByStorageTypeAndStatus generate slow query for multiple SD's
Summary: [scale][ovirt-3.6.z] - GetStorageConnectionsByStorageTypeAndStatus generate s...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: Database.Core
Version: 3.6.2
Hardware: x86_64
OS: Linux
unspecified
medium
Target Milestone: ovirt-3.6.7
: 3.6.7.3
Assignee: Allon Mureinik
QA Contact: guy chen
URL:
Whiteboard:
Depends On: 1306325
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-05-18 15:54 UTC by Allon Mureinik
Modified: 2016-07-04 12:31 UTC (History)
9 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of: 1306325
Environment:
Last Closed: 2016-07-04 12:31:54 UTC
oVirt Team: Storage
Embargoed:
amureini: ovirt-3.6.z?
rule-engine: planning_ack?
amureini: devel_ack+
rule-engine: testing_ack+


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 58451 0 ovirt-engine-3.6 MERGED core: storage_domain_shared_status SQL calculation 2016-06-02 12:26:45 UTC
oVirt gerrit 58535 0 ovirt-engine-3.6.7 MERGED core: storage_domain_shared_status SQL calculation 2016-06-02 13:50:59 UTC

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.


Note You need to log in before you can comment on or make changes to this bug.