Eli Mesika has submitted this change and it was merged.
Change subject: core: Add proxy agent support verification
......................................................................
core: Add proxy agent support verification
When a proxy Host is selected, there is no check if the
Host supports the requested PM agent.
If for example a Host H1 in Cluster C1 need fencing and has agent A1,
if we have Host H2 in Cluster C2 that does not support A1 and a Host
H3 in Cluster C3 that supports A1, the current implementation may
select H2 as a proxy, this will fail because A1 is not supported on
C2 and all proxy retries will be done on H2 while we have H3 that should
succeed to fence H1 since H3 is in C# which supports A1.
This patch adds a check to the proxy selection algorithm that insures
that the selected proxy supports the required PM agent.
In case of dual agents, support for both agents is required since in
concurrent PM devices we need both, and in sequential PM devices Primary
might fail and then Secondary PM agent should attempt to fence the Host.
Change-Id: I6e6610e1e7be660c894c879288d9df9b0956eda5
Bug-Url:
https://bugzilla.redhat.com/show_bug.cgi?id=1036365
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/utils/src/main/java/org/ovirt/engine/core/utils/pm/VdsFenceOptions.java
2 files changed, 40 insertions(+), 8 deletions(-)
Approvals:
Eli Mesika: Verified; Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/22423
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I6e6610e1e7be660c894c879288d9df9b0956eda5
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server