Change in ovirt-engine[master]: engine: fix execution context of SetNonOperationalVdsCommand

[masayag at redhat.com]

Moti Asayag has submitted this change and it was merged.

Change subject: engine: fix execution context of SetNonOperationalVdsCommand
......................................................................


engine: fix execution context of SetNonOperationalVdsCommand

This patch sets the execution context of SetNonOperationalVdsCommand to
async when it triggers migrations for the VMs which are running in the
VDS.

Previously the execution context of this command was always sync which
caused to DataIntegrityViolationException in the following scenarion:
1. the SetNonOperationalVdsCommand was triggered
2. migrations for the VMs running in the Vds was triggered
3. the sync part of SetNonOperationalVdsCommand ended succesfully
causing the job to be marked with Finished status
4. the job is cleared from the db by the periodic process that clean
finished jobs
5. the migration fails, thus it's being rerun
6. the migration rerun process is trying to add step, but its job
doesn't exist in the DB

Change-Id: If615a9d9fdad324d535709e76b5cd9a5f94953c9
Bug-Url: https://bugzilla.redhat.com/888199
Signed-off-by: Arik Hadas <ahadas at 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
2 files changed, 24 insertions(+), 5 deletions(-)

Approvals:
  Moti Asayag: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/11207
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: If615a9d9fdad324d535709e76b5cd9a5f94953c9
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>