summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorOla Aunrønning <olaa@verizonmedia.com>2021-04-20 01:26:05 +0200
committerGitHub <noreply@github.com>2021-04-20 01:26:05 +0200
commitd5a27e853336924c1d9f74e4504abf1bf6aa1a70 (patch)
treef0c106442c4ed14a6dc7ef97ec2db96212929a29
parent739eb8295cd1c7010c23ceca01239b72106133ba (diff)
parentf8a8bec612e71e19b6e48859f4cb51cae6a27d06 (diff)
Merge pull request #17489 from vespa-engine/olaa/capacity-checker-dont-throw-on-unknown-hosts
Don't throw IllegalArgumentException if CapacityChecker receives unknown hosts
-rw-r--r--node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/CapacityChecker.java10
1 files changed, 1 insertions, 9 deletions
diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/CapacityChecker.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/CapacityChecker.java
index 2b48ae98549..3fd07e1db2a 100644
--- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/CapacityChecker.java
+++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/CapacityChecker.java
@@ -64,18 +64,10 @@ public class CapacityChecker {
}
public List<Node> nodesFromHostnames(List<String> hostnames) {
- List<Node> nodes = hostnames.stream().filter(nodeMap::containsKey)
+ return hostnames.stream().filter(nodeMap::containsKey)
.map(nodeMap::get)
.collect(Collectors.toList());
- if (nodes.size() != hostnames.size()) {
- Set<String> notFoundNodes = new HashSet<>(hostnames);
- notFoundNodes.removeAll(nodes.stream().map(Node::hostname).collect(Collectors.toList()));
- throw new IllegalArgumentException(String.format("Host(s) not found: [ %s ]",
- String.join(", ", notFoundNodes)));
- }
-
- return nodes;
}
public Optional<HostFailurePath> findHostRemovalFailure(List<Node> hostsToRemove) {