Omer Frenkel has submitted this change and it was merged.
Change subject: core: vmsMonitoring - persist vms data in refresh vms phase
......................................................................
core: vmsMonitoring - persist vms data in refresh vms phase
the new structure of the host-vms monitoring caused the save vms info to
db to be called later in the cycle than it used to be.
this caused issues in operations that executed in afterVMsRefreshTreatment, that
assumed the db is updated.
in this patch the save-to-db moved back into the refreshVmStats from the
afterVMsRefreshTreatment, also with processExternallyManagedVms and
processVmsWithDevicesChange that originally happened before the save to
db.
Change-Id: I5e69b0ba20588db55bb40e9ba6ca5472c3efe687
Signed-off-by: Omer Frenkel <ofrenkel(a)redhat.com>
---
M
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VmsMonitoring.java
1 file changed, 10 insertions(+), 11 deletions(-)
Approvals:
Shmuel Leib Melamud: Looks good to me, but someone else must approve
Omer Frenkel: Verified
Arik Hadas: Looks good to me, approved
--
To view, visit
https://gerrit.ovirt.org/39406
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I5e69b0ba20588db55bb40e9ba6ca5472c3efe687
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server