Change in ovirt-engine[ovirt-engine-3.5]: core: fixing fencing flow for blocked proxy

tnisan at redhat.com tnisan at redhat.com
Wed Sep 10 12:47:47 UTC 2014


Tal Nisan has submitted this change and it was merged.

Change subject: core: fixing fencing flow for blocked proxy
......................................................................


core: fixing fencing flow for blocked proxy

This patch fixes the following scenario :

DC1 with H1 (with PM) and H2 on cluster C1
another host H3 on DC1 cluster C2

When we are blocking the communication with iptables from H2 to H1 PM
card and use the default proxy preferences (cluster, dc) a Restart
operation will always fail.

The fix relies on the return code from the script that was executed by
VDSM rather than the proceedProxyReturnValue call that throws an
exception and also set the execution result according to that code.

In addition a missing initialization of the 'count' variable in the
'for' loop has been added.

This BZ depends also in a VDSM fix that makes stop/start operation sync
instead of fire-and-forget.

Change-Id: I70d35967554c88c943c4653a26642e6e21cc54d2
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1129381
Signed-off-by: Eli Mesika <emesika at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/FenceExecutor.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/FenceVdsVDSCommand.java
2 files changed, 3 insertions(+), 3 deletions(-)

Approvals:
  Eli Mesika: Verified
  Oved Ourfali: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I70d35967554c88c943c4653a26642e6e21cc54d2
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list