Change in ovirt-engine[master]: core: stop querying statistics from db in vms monitoring
ahadas at redhat.com
ahadas at redhat.com
Wed Jul 13 07:24:43 UTC 2016
Arik Hadas has submitted this change and it was merged.
Change subject: core: stop querying statistics from db in vms monitoring
......................................................................
core: stop querying statistics from db in vms monitoring
Now that we have VmStatistics cached in VmManager, we can use it in the
monitoring. So this patch removes the statistics data from the vms
monitoring view (vms_monitoring_view) and changes VmAnalyzer to use the
statistics that are stored in VmManager instead.
Change-Id: I8db3001119c751966b501cf994165dd994d44877
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDaoImpl.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VmManager.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmAnalyzer.java
M backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/monitoring/VmAnalyzerTest.java
M packaging/dbscripts/create_views.sql
5 files changed, 16 insertions(+), 34 deletions(-)
Approvals:
Shahar Havivi: Looks good to me, approved
Jenkins CI: Passed CI tests
Arik Hadas: Verified
--
To view, visit https://gerrit.ovirt.org/60342
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I8db3001119c751966b501cf994165dd994d44877
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Shahar Havivi <shavivi at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
More information about the Engine-commits
mailing list