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@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@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Ayal Baron <abaron@redhat.com> Gerrit-Reviewer: Liron Aravot <laravot@redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs@redhat.com>