Allon Mureinik has submitted this change and it was merged.
Change subject: vdsbroker: read vmJobs update from getVmStats call
......................................................................
vdsbroker: read vmJobs update from getVmStats call
For Live Merge, the vmStats return value may optionally contain a list
of vmJob entries. These jobs indicate new-style async tasks that run on
the VDS. As long as the job is returned in the stats, it is not
finished. Once the job is no longer reported, the caller should query
the expected end state to determine if the task was successful.
Change-Id: I6d0ab4372fe98f8d61f4f393e2823d278f33621f
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/vdsbroker/VdsBrokerObjectsBuilder.java
M
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsProperties.java
2 files changed, 57 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/26905
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I6d0ab4372fe98f8d61f4f393e2823d278f33621f
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: Yair Zaslavsky <yzaslavs(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server