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 <ofren...@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 <ofren...@redhat.com> Gerrit-Reviewer: Arik Hadas <aha...@redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skriva...@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com> Gerrit-Reviewer: Roy Golan <rgo...@redhat.com> Gerrit-Reviewer: Shmuel Leib Melamud <smela...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
