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 a call to AlertIfPowerManagementOperationSkipped method in
handleError method.
This will force raising an Alert each time a PM operation was supposed to
be executed and from some reason the fencing operation has been failed.

In the case of dual agents:
for concurrent - on Start , Alert will be generated if both agents
failed
                 on Stop , Alert will be generated if one agent failed
for sequential
                 on start/stop , Alert will be generated if both agents

Change-Id: If5aef618f1e1ee8ab282bad528218b15b61085bd
Signed-off-by: Eli Mesika <[email protected]>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/FenceVdsBaseCommand.java
1 file changed, 1 insertion(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/23/12923/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/FenceVdsBaseCommand.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/FenceVdsBaseCommand.java
index 7b962c5..f3e0642 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/FenceVdsBaseCommand.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/FenceVdsBaseCommand.java
@@ -395,6 +395,7 @@
             log.errorFormat("Failed to {0} VDS using {1} Power Management 
agent", getParameters().getAction()
                     .name()
                     .toLowerCase(), order.name());
+            
AlertIfPowerManagementOperationSkipped(getParameters().getAction().name());
             throw new VdcBLLException(VdcBllErrors.VDS_FENCE_OPERATION_FAILED);
         } else { // Fence operation was skipped because Host is already in the 
requested state.
             setStatus(lastStatus);


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: If5aef618f1e1ee8ab282bad528218b15b61085bd
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

Reply via email to