Laszlo Hornyak has uploaded a new change for review. Change subject: engine: log if the external policy unit is enabled but the proxy is disabled ......................................................................
engine: log if the external policy unit is enabled but the proxy is disabled log if the external policy unit is enabled but the proxy is disabled Change-Id: Idc379fced2a078db27cffee3e261ac03142c73c8 Signed-off-by: Laszlo Hornyak <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/SchedulingManager.java 1 file changed, 5 insertions(+), 3 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/82/18282/1 diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/SchedulingManager.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/SchedulingManager.java index eddf180..ab5276e 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/SchedulingManager.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/SchedulingManager.java @@ -596,11 +596,13 @@ ClusterPolicy policy = policyMap.get(cluster.getClusterPolicyId()); PolicyUnitImpl policyUnit = policyUnits.get(policy.getBalance()); Pair<List<Guid>, Guid> balanceResult = null; - if (policyUnit.isInternal()){ + if (policyUnit.isInternal()) { balanceResult = internalRunBalance(policyUnit, cluster); - } else if (Config.GetValue(ConfigValues.ExternalSchedulerEnabled)) { - if (policyUnit.isEnabled()) { + } else if (policyUnit.isEnabled()) { + if (Config.GetValue(ConfigValues.ExternalSchedulerEnabled)) { balanceResult = externalRunBalance(policyUnit, cluster); + } else { + log.warn("Policy is enabled while external scheduler is disabled"); } } -- To view, visit http://gerrit.ovirt.org/18282 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Idc379fced2a078db27cffee3e261ac03142c73c8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Laszlo Hornyak <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
