Change in ovirt-engine[master]: core: AttachStorageDomainToPoolCommand - dont pass compensat...

laravot at redhat.com laravot at redhat.com
Mon Aug 8 06:29:03 UTC 2016


Liron Aravot has submitted this change and it was merged.

Change subject: core: AttachStorageDomainToPoolCommand - dont pass compensation context
......................................................................


core: AttachStorageDomainToPoolCommand - dont pass compensation context

If the data center is unintnalized, AttachStorageDomainToPoolCommand 
executes the AddStoragePoolWithStorages command. Currently the 
compensation context is passed to that command, on failure during the 
command execution the data registered on the compensation context won't 
be reverted becasue of an infrastractural bug.

Regardless of the need to fix the infrastractual issue in bz 1364804, 
this patch  solves the bug by not passing the compensation context

Change-Id: I80c56009b005917bdb2b4a642fadef32623c59d0
Bug-Url: https://bugzilla.redhat.com/1359788
Signed-off-by: Liron Aravot <laravot at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/domain/AttachStorageDomainToPoolCommand.java
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
  Jenkins CI: Passed CI tests
  Allon Mureinik: Looks good to me, approved
  Liron Aravot: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I80c56009b005917bdb2b4a642fadef32623c59d0
Gerrit-PatchSet: 3
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: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list