Change in ovirt-engine[master]: core: fix fence-status operation

[oourfali at redhat.com]

Oved Ourfali has submitted this change and it was merged.

Change subject: core: fix fence-status operation
......................................................................


core: fix fence-status operation

When the fencing policy is null, we check that the host is compatible
with cluster level 3.0, which is wrong. This patch changes that by
not checking cluster level when fencing policy is null. Also
getMinimalSupportedVersion was modified to throw
IllegalArgumentException when null is passed as parameter.

Change-Id: Idecf91da729cbd8744f894c341b7044cc791853d
Bug-Url: https://bugzilla.redhat.com/1130791
Signed-off-by: Oved Ourfali <oourfali at redhat.com>
Signed-off-by: Martin Perina <mperina at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/FenceExecutor.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/utils/FencingPolicyHelper.java
2 files changed, 17 insertions(+), 11 deletions(-)

Approvals:
  Martin Peřina: Verified
  Oved Ourfali: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Idecf91da729cbd8744f894c341b7044cc791853d
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Oved Ourfali <oourfali 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