Omer Frenkel has submitted this change and it was merged. Change subject: core: fix monitoring of SetNonOperationalVdsCommand ......................................................................
core: fix monitoring of SetNonOperationalVdsCommand This patch resolves a bug in which an exception was thrown when a step for an already completed and cleaned-up job is saved, when SetNonOperationalVdsCommand is invoked and finished and then a step for a migration it triggered is about to save. The solution is to extract the migrations to be monitored as different job, which reflects the process better on this scenario as the host become non-oprational independently of the status of the triggered migrations. Change-Id: I06b08108b1c75fe00b3f4dc6336c61a5e8c95d85 Bug-Url: https://bugzilla.redhat.com/888199 Signed-off-by: Arik Hadas <aha...@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/common/src/main/java/org/ovirt/engine/core/common/action/MigrateVmParameters.java M backend/manager/modules/dal/src/main/resources/bundles/ExecutionMessages.properties 4 files changed, 20 insertions(+), 4 deletions(-) Approvals: Omer Frenkel: Looks good to me, approved Arik Hadas: Verified Moti Asayag: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/11370 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I06b08108b1c75fe00b3f4dc6336c61a5e8c95d85 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <aha...@redhat.com> Gerrit-Reviewer: Arik Hadas <aha...@redhat.com> Gerrit-Reviewer: Moti Asayag <masa...@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com> Gerrit-Reviewer: Roy Golan <rgo...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
