Change in ovirt-engine[master]: vdsbroker: update vmJobs when updating vm statistics

[amureini at redhat.com]

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 at 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 at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadgett at redhat.com>
Gerrit-Reviewer: Liron Ar <laravot at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Ravi Nori <rnori at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: automation at ovirt.org