Allon Mureinik has submitted this change and it was merged.
Change subject: vdsbroker: update vmJobs when updating vm statistics
......................................................................
vdsbroker: update vmJobs when updating vm statistics
Update vmJob data in the database when updating vmStats. The jobs
records are created by the initiator of the job (i.e. a command), and
then may be updated and/or removed by VdsUpdateRunTimeInfo. The job
should be run before the job placeholder is inserted into the database
in order to limit the possibility of a query being made after the
insertion but before job execution, which would result in the job being
prematurely removed from the database.
Change-Id: If1a300d440a0a93381afe047eb294106a38088ec
Bug-Url:
https://bugzilla.redhat.com/1058160
Signed-off-by: Greg Padgett <gpadgett(a)redhat.com>
---
M
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java
1 file changed, 77 insertions(+), 0 deletions(-)
Approvals:
Ravi Nori: Looks good to me, but someone else must approve
Greg Padgett: Verified
Allon Mureinik: Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/26906
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: If1a300d440a0a93381afe047eb294106a38088ec
Gerrit-PatchSet: 14
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Greg Padgett <gpadgett(a)redhat.com>
Gerrit-Reviewer: Adam Litke <alitke(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com>
Gerrit-Reviewer: Liron Ar <laravot(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org