Change in ovirt-engine[master]: core: domain might remain locked on deactivation

laravot at redhat.com laravot at redhat.com
Thu Jan 22 15:07:50 UTC 2015


Liron Aravot has submitted this change and it was merged.

Change subject: core: domain might remain locked on deactivation
......................................................................


core: domain might remain locked on deactivation

When passing the compensation context to the child command the
'recorded' changes aren't reverted becasue when attempting to load the
compensation data the load is by the command id which is different
between the parent command and the child command.

When no tasks are created when deactivating domain and the actual
deactivation fails, the domain remained locked because of the above
reason.

Change-Id: I4a1bea6522ceb8c83be9466ed50748559745c3c8
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1184927
Signed-off-by: Liron Aravot <laravot at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/DeactivateStorageDomainWithOvfUpdateCommand.java
1 file changed, 2 insertions(+), 3 deletions(-)

Approvals:
  Allon Mureinik: Looks good to me, approved
  Liron Aravot: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4a1bea6522ceb8c83be9466ed50748559745c3c8
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list