Change in ovirt-engine[master]: core: avoid having SD active on vdsm and not in engine

[derez at redhat.com]

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 at 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 at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Michael Kublin <mkublin at redhat.com>
Gerrit-Reviewer: Vered Volansky <vvolansk at redhat.com>