
Omer Frenkel has submitted this change and it was merged. Change subject: core: keep engine lock after execute stage for async commands ...................................................................... core: keep engine lock after execute stage for async commands The engine lock is automatically released when the command ends the execute method with no tasks, even if the command is set to keep the locks for the whole command lifecycle. It should be that way for most of the commands, but not for async commands (which implement IVdsAsyncCommand) because such command always end the execute method with no tasks and we don't want their locks to be released after its execution when they are set to keep their lock for the entire execution. Thus this patch adds a check that prevents the engine lock from being released after the execute method for async commands. Change-Id: I5f7ef28a1ddef613b457aa47a85d9c0b5ce7a8d7 Bug-Url: https://bugzilla.redhat.com/952147 Signed-off-by: Arik Hadas <ahadas@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Arik Hadas: Verified Yair Zaslavsky: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18470 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5f7ef28a1ddef613b457aa47a85d9c0b5ce7a8d7 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server