diff options
author | Jon Marius Venstad <jonmv@users.noreply.github.com> | 2019-11-19 13:36:32 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-11-19 13:36:32 +0100 |
commit | e8c2faeb2c1feac0a3712592f4a55ce276d2fc60 (patch) | |
tree | 749a697db19e26e35d9aca048d726f4af0d597b7 | |
parent | 957a7cc70ba85568618fb2b5282d38f009c688ea (diff) | |
parent | 3ffe5f3f701a751415ceb6ac3911fb73891cb888 (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.java | 1 |
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; |