Change in ovirt-engine[ovirt-engine-3.5]: core: fix fence-status operation
Tal Nisan 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(a)redhat.com>
Signed-off-by: Martin Perina <mperina(a)redhat.com>
(cherry picked from commit 7039195f1019371a1edbcf3b97a82a098e77c868)
---
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: Looks good to me, but someone else must approve
Oved Ourfali: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/31603
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Idecf91da729cbd8744f894c341b7044cc791853d
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Martin Peřina <mperina(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