Eli Mesika has submitted this change and it was merged. Change subject: core:vm migration in case of failure ...(#818289) ...................................................................... core:vm migration in case of failure ...(#818289) vm migration in case of failure is not logged in task manager https://bugzilla.redhat.com/show_bug.cgi?id=818289 When we have Host A and Host B where Host A is SPM and we have HA VM running on Host A, if Host A is going to non-operational (can not access the storage for example) the HA VM will be migrated to Host B and Host B will become the SPM but we will not see the Migration under the SetNonOperational job. The reason is that the SetNonOperational ExceutionContext is not passed to MigrateAllVms method in order to use this context as the parent job context and to associate all migration steps with the SetNonOperational job. This patch changes the MigrateAllVms call to pass the parent SetNonOperational job ExceutionContext, this context is used as the parent ExceutionContext when migration steps are created for each VM. Change-Id: I437893ccf373af863810696d14ce8fd1ffb0253b Signed-off-by: Eli Mesika <emesika@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MaintananceVdsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/SetNonOperationalVdsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsEventListener.java 3 files changed, 12 insertions(+), 12 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6148 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I437893ccf373af863810696d14ce8fd1ffb0253b Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika@redhat.com> Gerrit-Reviewer: Eli Mesika <emesika@redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin@redhat.com> Gerrit-Reviewer: Moti Asayag <masayag@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs@redhat.com>