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