Bug 992883 - VdsManager.OnTimer loads VDS instead of VdsDynamic
Summary: VdsManager.OnTimer loads VDS instead of VdsDynamic
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: oVirt
Classification: Retired
Component: ovirt-engine-core
Version: unspecified
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: 3.3.2
Assignee: Liran Zelkha
QA Contact:
URL:
Whiteboard: infra
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-08-05 06:45 UTC by Liran Zelkha
Modified: 2023-09-14 01:48 UTC (History)
6 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2013-12-19 14:24:11 UTC
oVirt Team: ---
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 21662 0 None ABANDONED engine: Make VDSUpdateRuntimeInfo load partial VDS object Never

Description Liran Zelkha 2013-08-05 06:45:16 UTC
Description of problem:
VdsManager.OnTimer is triggered often (every 3-4 seconds) for each Vds. During the OnTimer method, an instance of VDS is loaded using a slow performing database query. 

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


How reproducible:
Run ovirt-engine and connect with a profiler to the code.

Steps to Reproduce:
1.
2.
3.

Actual results:
select from VDS view

Expected results:
select from VdsDynamic table.

Additional info:
VdsManager and VdsUpdateRuntimeInfo mostly use the VdsDynamic data, and do not require the complete VDS object. This will greatly improve database performance.

Comment 1 Liran Zelkha 2013-08-27 08:40:38 UTC
see http://gerrit.ovirt.org/#/c/17276/ for the fix

Comment 2 Sandro Bonazzola 2013-12-13 16:01:49 UTC
rc build done, moving to ON_QA, packages will be available on updates-testing repository.

Comment 3 Sandro Bonazzola 2013-12-19 14:24:11 UTC
oVirt 3.3.2 has been released resolving the problem described in this bug report.

Comment 4 Gilad Chaplik 2014-03-21 01:03:03 UTC
where is the fix?

Comment 5 Red Hat Bugzilla 2023-09-14 01:48:43 UTC
The needinfo request[s] on this closed bug have been removed as they have been unresolved for 1000 days


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