Eli Mesika has uploaded a new change for review. Change subject: core:Power Management or fence errors are not... ......................................................................
core:Power Management or fence errors are not... Power Management or fence errors are not logged in EVENT tab of GUI. Adding missing handling in VdsNotRespondingTreatmentCommand::canDoAction to log the failure as an alert if validation of this command fail as a result of a PM that was not configured properly. (Actually removing the "if" that did that only if Host PM is not configured at all...) Change-Id: Ic6aac13138c33f2631a1ea424428b4785c1d5700 Signed-off-by: Eli Mesika <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsNotRespondingTreatmentCommand.java 1 file changed, 1 insertion(+), 3 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/10/13510/1 diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsNotRespondingTreatmentCommand.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsNotRespondingTreatmentCommand.java index 39eee39..1ddd3cd 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsNotRespondingTreatmentCommand.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsNotRespondingTreatmentCommand.java @@ -74,9 +74,7 @@ } _vmsMovedToUnknown = true; log.errorFormat("Failed to run Fence script on vds:{0}, VMs moved to UnKnown instead.", getVdsName()); - if (!getVds().getpm_enabled()) { - AlertIfPowerManagementOperationSkipped(RESTART); - } + AlertIfPowerManagementOperationSkipped(RESTART); } @Override -- To view, visit http://gerrit.ovirt.org/13510 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ic6aac13138c33f2631a1ea424428b4785c1d5700 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
