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