Change in ovirt-engine[master]: core: skip failing proxy when waiting for status

oourfali at redhat.com oourfali at redhat.com
Tue Nov 4 10:52:55 UTC 2014


Oved Ourfali has submitted this change and it was merged.

Change subject: core: skip failing proxy when waiting for status
......................................................................


core: skip failing proxy when waiting for status

This patch fix a bug when we have a preferred proxy (in 6the same
cluster as the fenced host) that failed to perform the desired
operation.

In this case, we should remember the failed proxy inside the "get
status" loop and use the one that had succeeded (if exists)

Change-Id: I2b846444ff795e7099b61f6d0be8909402b7bfa1
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1141514
Signed-off-by: Eli Mesika <emesika at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/FenceExecutor.java
1 file changed, 17 insertions(+), 16 deletions(-)

Approvals:
  Martin Peřina: Looks good to me, but someone else must approve
  Eli Mesika: Verified
  Oved Ourfali: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I2b846444ff795e7099b61f6d0be8909402b7bfa1
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list