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