diff options
author | Ola Aunrønning <olaa@verizonmedia.com> | 2021-04-20 01:26:05 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-04-20 01:26:05 +0200 |
commit | d5a27e853336924c1d9f74e4504abf1bf6aa1a70 (patch) | |
tree | f0c106442c4ed14a6dc7ef97ec2db96212929a29 /node-repository | |
parent | 739eb8295cd1c7010c23ceca01239b72106133ba (diff) | |
parent | f8a8bec612e71e19b6e48859f4cb51cae6a27d06 (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
Diffstat (limited to 'node-repository')
-rw-r--r-- | node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/CapacityChecker.java | 10 |
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) { |