aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJon Marius Venstad <jonmv@users.noreply.github.com>2019-11-19 13:36:32 +0100
committerGitHub <noreply@github.com>2019-11-19 13:36:32 +0100
commite8c2faeb2c1feac0a3712592f4a55ce276d2fc60 (patch)
tree749a697db19e26e35d9aca048d726f4af0d597b7
parent957a7cc70ba85568618fb2b5282d38f009c688ea (diff)
parent3ffe5f3f701a751415ceb6ac3911fb73891cb888 (diff)
Merge pull request #11347 from vespa-engine/jvenstad/do-not-rebalance-tester-nodes
Skip tester owned nodes when considering best moves
-rw-r--r--node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/Rebalancer.java1
1 files changed, 1 insertions, 0 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 ffc4461af51..2fa7341d272 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
@@ -99,6 +99,7 @@ public class Rebalancer extends Maintainer {
Move bestMove = Move.none;
for (Node node : allNodes.nodeType(NodeType.tenant).state(Node.State.active)) {
if (node.parentHostname().isEmpty()) continue;
+ if (node.allocation().get().owner().instance().isTester())) continue;
for (Node toHost : allNodes.nodeType(NodeType.host).state(NodePrioritizer.ALLOCATABLE_HOST_STATES)) {
if (toHost.hostname().equals(node.parentHostname().get())) continue;
if ( ! capacity.freeCapacityOf(toHost).satisfies(node.flavor().resources())) continue;