Piotr Kliczewski 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(a)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/62041
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I80c56009b005917bdb2b4a642fadef32623c59d0
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.0
Gerrit-Owner: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>