Eli Mesika has uploaded a new change for review. Change subject: core: misleading error when trying to fen(#843168) ......................................................................
core: misleading error when trying to fen(#843168) ovirt-engine-backend: misleading error when trying to fence host without PM configured https://bugzilla.redhat.com/843168 Problem was that the code in the canDoAction of the base class was not adding any message if fencing is not configured in the host and also retries the operation event though there is no chance that it will succeed. Changing the code to add a message when Start/Stop is called on a non configured Host. Make StopVdsCommand call the super.canDoAction in the same manner as StartVdsCommand does. Change-Id: I7aab4f6a7c70fa5e18db106ea15a7b20b709db28 Signed-off-by: Eli Mesika <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/FenceVdsBaseCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/StopVdsCommand.java 2 files changed, 10 insertions(+), 9 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/79/6879/1 -- To view, visit http://gerrit.ovirt.org/6879 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I7aab4f6a7c70fa5e18db106ea15a7b20b709db28 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
