Arik Hadas has submitted this change and it was merged.
Change subject: core: fetch vm dynamic info from vm analyzers
......................................................................
core: fetch vm dynamic info from vm analyzers
Change the way VmsMonitoring fetches VM dynamic info from VM
analyzers: iterate the analyzers using the stream API and get
the dynamic info instead of the statistics being set inside
VmsMonitoring by the analyzers.
Note that the added code for processVmsWithDevicesChange is
not as efficient as it used to be, but the process of extracting
the handling of the devices from VmsMonitoring is in progress
and is expected to change this code anyway.
Change-Id: I4aa8e5a2778c0aae85644043b3a1530970a862eb
Signed-off-by: Arik Hadas <ahadas(a)redhat.com>
---
M
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VmAnalyzer.java
M
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VmsMonitoring.java
M
backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/VmAnalyzerTest.java
3 files changed, 29 insertions(+), 25 deletions(-)
Approvals:
Shahar Havivi: Looks good to me, approved
Jenkins CI: Passed CI tests
Shmuel Leib Melamud: Looks good to me, but someone else must approve
Arik Hadas: Verified
--
To view, visit
https://gerrit.ovirt.org/51692
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I4aa8e5a2778c0aae85644043b3a1530970a862eb
Gerrit-PatchSet: 11
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com>
Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>