summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHåkon Hallingstad <hakon@verizonmedia.com>2021-08-16 17:51:08 +0200
committerHåkon Hallingstad <hakon@verizonmedia.com>2021-08-16 17:51:08 +0200
commit9faa27eadc810013d55c31f7766a35c292f9dc52 (patch)
treeedbea095224ca24593f7b73ea93c6d9410530379
parentdd01522678083a224fab06fdac076e9c04e6042b (diff)
Remove unnecessary code
-rw-r--r--node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/NodeFailerTest.java6
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));