diff options
author | Martin Polden <mpolden@mpolden.no> | 2020-03-09 14:32:43 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-03-09 14:32:43 +0100 |
commit | 444bf1a659cbe6cb4b8558735158e901a6d9f395 (patch) | |
tree | f8278649cb810f800b93702c1e7154facdd6ae61 /node-repository/src | |
parent | e58c309c1efc4028bf853c41004ddf369546c7fb (diff) |
Revert "Remove inactive load balancers for combined cluster"
Diffstat (limited to 'node-repository/src')
-rw-r--r-- | node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/LoadBalancerExpirer.java | 11 |
1 files changed, 1 insertions, 10 deletions
diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/LoadBalancerExpirer.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/LoadBalancerExpirer.java index 93f0b75dcdf..e2b70608d58 100644 --- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/LoadBalancerExpirer.java +++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/LoadBalancerExpirer.java @@ -1,7 +1,6 @@ // Copyright 2018 Yahoo Holdings. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. package com.yahoo.vespa.hosted.provision.maintenance; -import com.yahoo.config.provision.ClusterSpec; import com.yahoo.log.LogLevel; import com.yahoo.vespa.hosted.provision.Node; import com.yahoo.vespa.hosted.provision.NodeRepository; @@ -131,15 +130,7 @@ public class LoadBalancerExpirer extends Maintainer { } private List<Node> allocatedNodes(LoadBalancerId loadBalancer) { - return nodeRepository().list() - .owner(loadBalancer.application()) - .cluster(loadBalancer.cluster()) - // TODO(mpolden): Remove after March 2020. This ignores nodes in combined clusters so - // that inactive load balancers for combined clusters with wrong ID are - // eventually removed. - .filter(node -> node.allocation().isPresent() && - node.allocation().get().membership().cluster().type() != ClusterSpec.Type.combined) - .asList(); + return nodeRepository().list().owner(loadBalancer.application()).cluster(loadBalancer.cluster()).asList(); } } |