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(a)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(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server