diff options
author | Håkon Hallingstad <hakon@verizonmedia.com> | 2021-08-16 17:51:08 +0200 |
---|---|---|
committer | Håkon Hallingstad <hakon@verizonmedia.com> | 2021-08-16 17:51:08 +0200 |
commit | 9faa27eadc810013d55c31f7766a35c292f9dc52 (patch) | |
tree | edbea095224ca24593f7b73ea93c6d9410530379 /node-repository | |
parent | dd01522678083a224fab06fdac076e9c04e6042b (diff) |
Remove unnecessary code
Diffstat (limited to 'node-repository')
-rw-r--r-- | node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/NodeFailerTest.java | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/NodeFailerTest.java b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/NodeFailerTest.java index 0f0da628d13..2bd0c91f4a1 100644 --- a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/NodeFailerTest.java +++ b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/NodeFailerTest.java @@ -592,18 +592,14 @@ public class NodeFailerTest { // 10 hosts with 3 tenant nodes each, total 40 nodes NodeFailTester tester = NodeFailTester.withTwoApplications(10); NodeList hosts = tester.nodeRepository.nodes().list().nodeType(NodeType.host); - // TODO: Remove deadNodes - List<Node> deadNodes = new ArrayList<>(); // 3 hosts fail. 2 of them and all of their children are allowed to fail List<Node> failedHosts = hosts.asList().subList(0, 3); failedHosts.forEach(host -> { tester.serviceMonitor.setHostDown(host.hostname()); - deadNodes.add(host); }); tester.runMaintainers(); tester.clock.advance(Duration.ofMinutes(61)); - tester.allNodesMakeAConfigRequestExcept(deadNodes); tester.runMaintainers(); assertEquals(2 + /* hosts */ @@ -615,7 +611,6 @@ public class NodeFailerTest { // 24 more hours pass without any other nodes being failed out for (int minutes = 0, interval = 30; minutes <= 23 * 60; minutes += interval) { tester.clock.advance(Duration.ofMinutes(interval)); - tester.allNodesMakeAConfigRequestExcept(deadNodes); } tester.runMaintainers(); assertEquals(8, tester.nodeRepository.nodes().list(Node.State.failed).size()); @@ -631,7 +626,6 @@ public class NodeFailerTest { // Nothing else to fail tester.clock.advance(Duration.ofHours(25)); - tester.allNodesMakeAConfigRequestExcept(deadNodes); tester.runMaintainers(); assertEquals(12, tester.nodeRepository.nodes().list(Node.State.failed).size()); assertEquals("Throttling is not indicated by the metric", 0, tester.metric.values.get(NodeFailer.throttlingActiveMetric)); |