Change in ovirt-engine[master]: core: fetch vm dynamic info from vm analyzers

ahadas at redhat.com ahadas at redhat.com
Mon Jan 18 13:44:50 UTC 2016


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 at 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 at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Shahar Havivi <shavivi at redhat.com>
Gerrit-Reviewer: Shmuel Leib Melamud <smelamud at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list