Eli Mesika has uploaded a new change for review.

Change subject: core: skip non-responding hosts in proxy selection.
......................................................................

core: skip non-responding hosts in proxy selection.

non-responding host is selected as a proxy for fencing operations

This patch adds the host NonResponsive status as an illegal status for
the proxy selection algorithm.
Hosts that are in NonResponsive state have communication problems that
prevents them from being proxy candidates.

Change-Id: Idaa861b32b20cca43a7fe41f318c3e8bcec4c95d
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1145321
Signed-off-by: Eli Mesika <emes...@redhat.com>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/FenceExecutor.java
1 file changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/31/33431/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 7b7e883..a0c9da7 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
@@ -316,7 +316,8 @@
                  || vdsDynamic.getStatus() == VDSStatus.Reboot
                  || vdsDynamic.getStatus() == VDSStatus.Kdumping
                  || (vdsDynamic.getStatus() == VDSStatus.NonOperational
-                     && vdsDynamic.getNonOperationalReason() == 
NonOperationalReason.NETWORK_UNREACHABLE));
+                     && vdsDynamic.getNonOperationalReason() == 
NonOperationalReason.NETWORK_UNREACHABLE)
+                 || vdsDynamic.getStatus() == VDSStatus.NonResponsive);
     }
 
     private VDS getFenceProxy(final boolean onlyUpHost, final boolean 
filterSelf, final PMProxyOptions proxyOptions) {


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Idaa861b32b20cca43a7fe41f318c3e8bcec4c95d
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Eli Mesika <emes...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to