Change in ovirt-engine[master]: core: adding VdsmImagePoller

Code Review gerrit at ovirt.org
Mon Nov 21 09:04:55 UTC 2016


>From Liron Aravot <laravot at redhat.com>:

Liron Aravot has submitted this change and it was merged.

Change subject: core: adding VdsmImagePoller
......................................................................


core: adding VdsmImagePoller

This patch adds the VdsmImagePoller class. The poller is supposed to
be used to determine the status of ongoing vdsm job on vdsm image when
the job polling isn't possible.

The poller uses the following logic:
1. The volume lease isn't FREE - the operation is running
2. The lease is free and the volume status is ILLEGAL - the operation
has failed.
3. The lease is free, the volume status is OK and the generation was
incremented - the operation has succeded.

Any other case means that the job status couldn't have been determined
using entity polling.

Change-Id: Ibf3f4d27ece12b2d855cdf3c1f5a563019aa94cc
Signed-off-by: Liron Aravot <laravot at redhat.com>
---
A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/image/VdsmImagePoller.java
1 file changed, 56 insertions(+), 0 deletions(-)

Approvals:
  Tal Nisan: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Freddy Rolland: Looks good to me, but someone else must approve
  Liron Aravot: Verified



-- 
To view, visit https://gerrit.ovirt.org/66848
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ibf3f4d27ece12b2d855cdf3c1f5a563019aa94cc
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list