Eli Mesika has uploaded a new change for review.

Change subject: core:[RFE] Improve power management proxy select..
......................................................................

core:[RFE] Improve power management proxy select..

[RFE] Improve power management proxy selection

this patch resolves the following scenario:

- cluster-1
  - host-X
  - host-Y
- cluster-2
  - host-Z

- Restart host-Y - succeed
- Restart host-X - Fail: "General command validation failure."

The patch odes not consider a host as a proxy candidate if it is in the
Reboot state

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


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/24/12924/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/FenceExecutor.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/FenceExecutor.java
index 367f29c..08354c8 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/FenceExecutor.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/FenceExecutor.java
@@ -271,7 +271,7 @@
 
     private boolean isHostNetworkUnreacable(VDS vds) {
         VdsDynamic vdsDynamic = vds.getDynamicData();
-        return (vdsDynamic.getstatus() == VDSStatus.NonOperational && 
vdsDynamic.getNonOperationalReason() == 
NonOperationalReason.NETWORK_UNREACHABLE);
+        return ((vdsDynamic.getstatus() == VDSStatus.Reboot) || 
(vdsDynamic.getstatus() == VDSStatus.NonOperational && 
vdsDynamic.getNonOperationalReason() == 
NonOperationalReason.NETWORK_UNREACHABLE));
     }
 
     private VDS getFenceProxy(final boolean onlyUpHost, final boolean 
filterSelf, final PMProxyOptions proxyOptions) {


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

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