Change in ovirt-engine[master]: core: update interface statistics not thorugh resource manager

ahadas at redhat.com ahadas at redhat.com
Thu Jul 21 07:17:08 UTC 2016


Arik Hadas has submitted this change and it was merged.

Change subject: core: update interface statistics not thorugh resource manager
......................................................................


core: update interface statistics not thorugh resource manager

VmAnalyzer remained the only place that rely on the internalSetVmStatus
in ResourceManager to update VM network interface statistics. This
patch changes it so we could eventually remove this part from the
internalSetVmStatus method.

Change-Id: Ia9c02d7ec774ab444f84d87a0d6c8664eb49d62a
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/network/NetworkStatistics.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/ResourceManager.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
4 files changed, 15 insertions(+), 12 deletions(-)

Approvals:
  Shahar Havivi: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Arik Hadas: Verified



-- 
To view, visit https://gerrit.ovirt.org/60927
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia9c02d7ec774ab444f84d87a0d6c8664eb49d62a
Gerrit-PatchSet: 7
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: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list