Change in ovirt-engine[master]: core: fixing other_dc proxy preference implementation

oourfali at redhat.com oourfali at redhat.com
Mon Aug 25 09:35:52 UTC 2014


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 at 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 at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list