Change in ovirt-engine[master]: engine: Setting reconstruct to failed when last master was d...

[mkublin at redhat.com]

Michael Kublin has submitted this change and it was merged.

Change subject: engine: Setting reconstruct to failed when last master was disabled
......................................................................


engine: Setting reconstruct to failed when last master was disabled

The following commit is changing return value of reconstruct command to false
when we has only last master and it was changed to InActive status.
These fix is important for InitVdsOnUpCommand and also fixing an old bug - when a command
was marked as successed but audit log message was "Failed to ..."

Change-Id: I0e889d5e3022963bf835ea48674f23e80ca49675
Bug-Url: https://bugzilla.redhat.com/895996
Signed-off-by: Michael Kublin <mkublin at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ReconstructMasterDomainCommand.java
1 file changed, 4 insertions(+), 1 deletion(-)

Approvals:
  Michael Kublin: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/11101
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I0e889d5e3022963bf835ea48674f23e80ca49675
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Michael Kublin <mkublin at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Michael Kublin <mkublin at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>