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: Ie886cb47eb036aa0be8f0d6c7a8f4a6957ffb8f0
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, 4 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/81/18281/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..5ff2578 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
@@ -598,9 +598,11 @@
             Pair<List<Guid>, Guid> balanceResult = null;
             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/18281
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie886cb47eb036aa0be8f0d6c7a8f4a6957ffb8f0
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

Reply via email to