Oved Ourfali has submitted this change and it was merged.
Change subject: core: fixing other_dc proxy preference implementation
......................................................................
core: fixing other_dc proxy preference implementation
Fencing with proxy from other_dc is actually performed by a proxy in the
same dc
This patch actually issues two things
1) the selected proxy preferences where not passed by the HostModel so
always the "cluster,dc" default was treated
2) when searching for a proxy on other_dc, hosts from the fenced host DC
where not filtered out.
Change-Id: I0de81858dc9f9b76bd27c75cb47086c884f1d6cd
Bug-Url:
https://bugzilla.redhat.com/show_bug.cgi?id=1131411
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/bll/src/main/java/org/ovirt/engine/core/bll/GetNewVdsFenceStatusQuery.java
M
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/GetNewVdsFenceStatusParameters.java
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostModel.java
4 files changed, 19 insertions(+), 3 deletions(-)
Approvals:
Martin Peřina: Looks good to me, but someone else must approve
Eli Mesika: Verified
Gilad Chaplik: Looks good to me, but someone else must approve
Oved Ourfali: Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/31870
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I0de81858dc9f9b76bd27c75cb47086c884f1d6cd
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com>
Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server