Change in ovirt-engine[ovirt-engine-4.1]: engine: Detach RemoveImage from MoveDisk command

[Code Review]

>From Tal Nisan <tnisan at redhat.com>:

Tal Nisan has submitted this change and it was merged. ( https://gerrit.ovirt.org/78783 )

Change subject: engine: Detach RemoveImage from MoveDisk command
......................................................................


engine: Detach RemoveImage from MoveDisk command

At the end of the MoveImageGroupCommand, a RemoveImage command
is called to clean the source disk.

In the case the RemoveImage is taking long to finish, the Job
of the MoveImageGroupCommand could be cleared meanwhile and when
the RemoveImage command will try to update its step it will fail to
find the Job entry causing an SQL exception.

In order to avoid this situation, the RemoveImage command will
be executed in a new context.
Note that its execution will not be reported in the UI.

Change-Id: I86deaf1f3d716e28e76236c73f13c830b3c7338d
Bug-Url: https://bugzilla.redhat.com/1464348
Signed-off-by: Fred Rolland <frolland at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/image/MoveImageGroupCommand.java
1 file changed, 3 insertions(+), 2 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Allon Mureinik: Looks good to me, approved
  Freddy Rolland: Verified



-- 
To view, visit https://gerrit.ovirt.org/78783
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I86deaf1f3d716e28e76236c73f13c830b3c7338d
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.1
Gerrit-Owner: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Fred Rolland <frolland at redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>