Change in ovirt-engine[ovirt-engine-3.6]: engine: Completed task still shown as 'running' in UI

Oved Ourfali has submitted this change and it was merged. Change subject: engine: Completed task still shown as 'running' in UI ...................................................................... engine: Completed task still shown as 'running' in UI Currently when SPM host goes down due to connectivity issue, VdsNotRespondingTreatment command is left forever 'running' (hourglass icon) in the WebAdmin tasks pane. This happens becuase when SPM host goes down the datacenter should go down, and thus an internal command: SetStoragePoolStatus is invoked, and there is a mixup in the Execution-Context. This patch solves this bug locally, although a similar issue may exist in other places and handling of CommandContext and ExecutionContext should be revisited. Bug-Url: https://bugzilla.redhat.com/1145099 Change-Id: I4f5fbaef3e8c57fbad7997cb67282a6c163b6b03 Signed-off-by: Ori Liel <oliel@redhat.com> (cherry picked from commit 7714abd1dc71275cc109d57f1849dd9e68e7c6b2) --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/job/ExecutionContext.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/pm/VdsNotRespondingTreatmentCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/FenceValidator.java 3 files changed, 47 insertions(+), 26 deletions(-) Approvals: Ori Liel: Verified Jenkins CI: Passed CI tests Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/44595 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4f5fbaef3e8c57fbad7997cb67282a6c163b6b03 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Oved Ourfali <oourfali@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ori Liel <oliel@redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali@redhat.com> Gerrit-Reviewer: automation@ovirt.org
participants (1)
-
oourfali@redhat.com