Change in ovirt-engine[master]: core: avoid having SD active on vdsm and not in engine
Daniel Erez has submitted this change and it was merged.
Change subject: core: avoid having SD active on vdsm and not in engine
......................................................................
core: avoid having SD active on vdsm and not in engine
When running RecoveryStoragePool command which triggers reconstruct
engine shouldn't use compensation on the newly
added domain in case the reconstruct succeeded on vdsm side even if we
have a failure afterwards.
Change-Id: Ida51b027b8078555ffab5b80d2ffe54572fa6cca
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=845838
Signed-off-by: Liron Aravot <laravot(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ReconstructMasterDomainCommand.java
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/RecoveryStoragePoolCommand.java
2 files changed, 28 insertions(+), 16 deletions(-)
Approvals:
Daniel Erez: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/8390
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ida51b027b8078555ffab5b80d2ffe54572fa6cca
Gerrit-PatchSet: 8
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <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 Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com>
Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com>