From Liron Aravot <laravot(a)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(a)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(a)redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>