Change in ovirt-engine[master]: core: vms monitoring no longer always expects vm statistics

[ahadas at redhat.com]

Arik Hadas has submitted this change and it was merged.

Change subject: core: vms monitoring no longer always expects vm statistics
......................................................................


core: vms monitoring no longer always expects vm statistics

VmStatistics are processed and persisted only on statistics monitoring
cycles. However, while analyzing the VM we used to count on having the
statistics also for regular (non-statistics) monitoring cycles. This
required as to have a workaround in the handling of events, which do
not include the statistics (the statistics were loaded from the DB).

Now that we use VmStatistics data only on statistics monitoring cycles
we can remove this workaround.

Change-Id: If5d69f975b4245f0ae05ba4ccfec1fa24a57237f
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/EventVmStatsRefresher.java
1 file changed, 5 insertions(+), 6 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: If5d69f975b4245f0ae05ba4ccfec1fa24a57237f
Gerrit-PatchSet: 2
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>