summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJon Bratseth <bratseth@verizonmedia.com>2019-10-17 20:35:41 +0200
committerJon Bratseth <bratseth@verizonmedia.com>2019-10-17 20:35:41 +0200
commit79c303d743daefb80e5a4d4e9f12b2c41a7aa69d (patch)
tree6ec08e682c0297f0b75c5869f4171527bbef8bfc
parentb4689a871d31c1f47dd8ac8d6b4e1d16e1f8edd0 (diff)
Check node just once per iteration
-rw-r--r--node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/Rebalancer.java2
1 files changed, 1 insertions, 1 deletions
diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/Rebalancer.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/Rebalancer.java
index e8d0839d60e..42b51f521bc 100644
--- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/Rebalancer.java
+++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/Rebalancer.java
@@ -56,8 +56,8 @@ public class Rebalancer extends Maintainer {
DockerHostCapacity capacity = new DockerHostCapacity(allNodes, hostResourcesCalculator);
Move bestMove = Move.none;
for (Node node : allNodes.state(Node.State.active)) {
+ if (node.parentHostname().isEmpty()) continue;
for (Node toHost : allNodes.state(NodePrioritizer.ALLOCATABLE_HOST_STATES).nodeType(NodeType.host)) {
- if (node.parentHostname().isEmpty()) continue;
if (toHost.hostname().equals(node.parentHostname().get())) continue;
if ( ! capacity.freeCapacityOf(toHost).satisfies(node.flavor().resources())) continue;