Change in ovirt-engine[master]: core: Fix checking compatibility of proxy host and fencing p...

oourfali at redhat.com oourfali at redhat.com
Thu Sep 3 16:09:22 UTC 2015


Oved Ourfali has submitted this change and it was merged.

Change subject: core: Fix checking compatibility of proxy host and fencing policy
......................................................................


core: Fix checking compatibility of proxy host and fencing policy

Fixes the issue when checking compatibility of fence proxy host with
fencing policy caused by BZ1229177. It changes the comparison using
following rule: If at least one version contained in host supported
cluster levels is higher or equals to minimal required version for the
fencing policy of the cluster, then host can be used as a proxy.

Change-Id: I94cc1419186dfd3ab557143e41b3d3f3772f614a
Bug-Url: https://bugzilla.redhat.com/1257610
Signed-off-by: Martin Perina <mperina at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/pm/FenceProxyLocator.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/pm/FenceProxyLocatorTest.java
2 files changed, 25 insertions(+), 3 deletions(-)

Approvals:
  Martin Peřina: Verified
  Jenkins CI: Passed CI tests
  Oved Ourfali: Looks good to me, approved



-- 
To view, visit https://gerrit.ovirt.org/45725
To unsubscribe, visit https://gerrit.ovirt.org/settings

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



More information about the Engine-commits mailing list