Change in ovirt-engine[ovirt-engine-3.5]: core: fixing fencing flow for blocked proxy
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(a)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(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server