diff options
author | Håkon Hallingstad <hakon@yahooinc.com> | 2023-01-09 16:26:51 +0100 |
---|---|---|
committer | Håkon Hallingstad <hakon@yahooinc.com> | 2023-01-09 16:26:51 +0100 |
commit | 21de9943fb135a3cdde3957b4ace79be5edc55ec (patch) | |
tree | c92207ef2c22766bc13a62439114a81428e077bc /node-repository/src/test/java | |
parent | 6bd735e4b444bbcffe54778881f17e194ac47d79 (diff) |
Patch wantToDeprovision recursively
Diffstat (limited to 'node-repository/src/test/java')
2 files changed, 4 insertions, 5 deletions
diff --git a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/ProvisionedExpirerTest.java b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/ProvisionedExpirerTest.java index 32886733856..359f75c27ab 100644 --- a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/ProvisionedExpirerTest.java +++ b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/ProvisionedExpirerTest.java @@ -37,12 +37,12 @@ public class ProvisionedExpirerTest { tester.clock().advance(Duration.ofMinutes(5)); new ProvisionedExpirer(tester.nodeRepository(), Duration.ofMinutes(4), new TestMetric()).maintain(); - assertEquals(5, tester.nodeRepository().nodes().list().deprovisioning().size()); - assertEquals(20, tester.nodeRepository().nodes().list().not().deprovisioning().size()); + assertEquals(10, tester.nodeRepository().nodes().list().deprovisioning().size()); + assertEquals(5, tester.nodeRepository().nodes().list().not().deprovisioning().size()); } private void populateNodeRepo() { - var nodes = IntStream.range(0, 25) + var nodes = IntStream.range(0, 15) .mapToObj(i -> Node.create("id-" + i, "host-" + i, new Flavor(NodeResources.unspecified()), Node.State.provisioned, NodeType.host).build()) .toList(); tester.nodeRepository().database().addNodesInState(nodes, Node.State.provisioned, Agent.system); diff --git a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/NodesV2ApiTest.java b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/NodesV2ApiTest.java index 55b401ef4e8..f5fe0e6aafd 100644 --- a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/NodesV2ApiTest.java +++ b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/NodesV2ApiTest.java @@ -218,9 +218,8 @@ public class NodesV2ApiTest { assertResponse(new Request("http://localhost:8080/nodes/v2/node/dockerhost2.yahoo.com", Utf8.toBytes("{\"wantToDeprovision\": true, \"wantToRetire\": true}"), Request.Method.PATCH), "{\"message\":\"Updated dockerhost2.yahoo.com\"}"); - // Make sure that wantToRetire is applied recursively, but wantToDeprovision isn't tester.assertResponseContains(new Request("http://localhost:8080/nodes/v2/node/host5.yahoo.com"), - "\"wantToRetire\":true,\"preferToRetire\":false,\"wantToDeprovision\":false,"); + "\"wantToRetire\":true,\"preferToRetire\":false,\"wantToDeprovision\":true,"); assertResponse(new Request("http://localhost:8080/nodes/v2/node/dockerhost1.yahoo.com", Utf8.toBytes("{\"wantToRebuild\": true, \"wantToRetire\": true}"), Request.Method.PATCH), "{\"message\":\"Updated dockerhost1.yahoo.com\"}"); |