Change in ovirt-engine[master]: core: vmsMonitoring - persist vms data in refresh vms phase

[ofrenkel at redhat.com]

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 at 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 at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Shmuel Leib Melamud <smelamud at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server