Change in ovirt-engine[master]: core: Compensate to unknown status to avoid rollback to acti...
Allon Mureinik has submitted this change and it was merged.
Change subject: core: Compensate to unknown status to avoid rollback to active status.
......................................................................
core: Compensate to unknown status to avoid rollback to active status.
On deactivate storage domain command, the storage domain(s) status is being
saved in the compensation context so in case of failure - it will
be reverted to its original value.
In case of failure when executing DeactiveStorageDomainVDSCommand this failure
will propagate to DeactiveStorageDomainCommand and will cause to the compensation
saved data to be applied - which might override the changed statuses and return the
domain status to be active again.
In this change, the domain status is being saved in the compensation
context in unknown status rather then active, so in case of failure the
status would be reverted to unknown.
Change-Id: I9f8a27cbac711fa87db408a06fe22a7b3e9b2da9
Bug-Url: https://bugzilla.redhat.com/920694
Signed-off-by: Liron Aravot <laravot(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/DeactivateStorageDomainCommand.java
1 file changed, 2 insertions(+), 0 deletions(-)
Approvals:
Allon Mureinik: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/14767
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I9f8a27cbac711fa87db408a06fe22a7b3e9b2da9
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Ar <laravot(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Liron Ar <laravot(a)redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com>