You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by re...@apache.org on 2015/12/04 19:30:04 UTC

[1/2] git commit: updated refs/heads/master to 2093c33

Repository: cloudstack
Updated Branches:
  refs/heads/master 6dcfd668f -> 2093c33eb


CLOUDSTACK-9107: Description of global config agent.load.threshold and the actual behavior are not in sync
Made configuration parameter description and behavior in sync.


Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/04c44894
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/04c44894
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/04c44894

Branch: refs/heads/master
Commit: 04c44894b961cede7a917bcc78b153a0fc8ffd46
Parents: 7e902cd
Author: Koushik Das <ko...@apache.org>
Authored: Fri Dec 4 14:08:56 2015 +0530
Committer: Koushik Das <ko...@apache.org>
Committed: Fri Dec 4 14:08:56 2015 +0530

----------------------------------------------------------------------
 .../com/cloud/agent/manager/ClusteredAgentManagerImpl.java   | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/04c44894/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
----------------------------------------------------------------------
diff --git a/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java b/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
index f707781..f35a4ef 100644
--- a/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
+++ b/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
@@ -1411,14 +1411,12 @@ public class ClusteredAgentManagerImpl extends AgentManagerImpl implements Clust
                         final double managedHostsCount = allManagedRoutingAgents.size();
                         if (allHostsCount > 0.0) {
                             final double load = managedHostsCount / allHostsCount;
-                            if (load >= ConnectedAgentThreshold.value()) {
-                                s_logger.debug("Scheduling agent rebalancing task as the average agent load " + load + " is more than the threshold " +
-                                        ConnectedAgentThreshold.value());
+                            if (load > ConnectedAgentThreshold.value()) {
+                                s_logger.debug("Scheduling agent rebalancing task as the average agent load " + load + " is more than the threshold " + ConnectedAgentThreshold.value());
                                 scheduleRebalanceAgents();
                                 _agentLbHappened = true;
                             } else {
-                                s_logger.debug("Not scheduling agent rebalancing task as the averages load " + load + " is less than the threshold " +
-                                        ConnectedAgentThreshold.value());
+                                s_logger.debug("Not scheduling agent rebalancing task as the average load " + load + " has not crossed the threshold " + ConnectedAgentThreshold.value());
                             }
                         }
                     }


[2/2] git commit: updated refs/heads/master to 2093c33

Posted by re...@apache.org.
Merge pull request #1172 from koushik-das/CLOUDSTACK-9107

CLOUDSTACK-9107: Description of global config agent.load.threshold and the actual behavior are not in sync

Made configuration parameter description and behavior in sync.

* pr/1172:
  CLOUDSTACK-9107: Description of global config agent.load.threshold and the actual behavior are not in sync Made configuration parameter description and behavior in sync.

Signed-off-by: Remi Bergsma <gi...@remi.nl>


Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/2093c33e
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/2093c33e
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/2093c33e

Branch: refs/heads/master
Commit: 2093c33ebd2a5641c7359f21b1df6b83acf82121
Parents: 6dcfd66 04c4489
Author: Remi Bergsma <gi...@remi.nl>
Authored: Fri Dec 4 19:29:17 2015 +0100
Committer: Remi Bergsma <gi...@remi.nl>
Committed: Fri Dec 4 19:29:17 2015 +0100

----------------------------------------------------------------------
 .../com/cloud/agent/manager/ClusteredAgentManagerImpl.java   | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)
----------------------------------------------------------------------