diff options
author | Martin Polden <mpolden@mpolden.no> | 2021-04-28 14:42:31 +0200 |
---|---|---|
committer | Martin Polden <mpolden@mpolden.no> | 2021-04-28 14:54:58 +0200 |
commit | add16758b8048ace856957ca4b9a412cc19aa471 (patch) | |
tree | 3e0cf62ce012b014b4af60208b6a36802dd9ec78 /node-repository/src/test/java | |
parent | 3894db1f1fb84d94ef5d89b703d8501c5205d8f9 (diff) |
Always use transaction when removing nodes
Diffstat (limited to 'node-repository/src/test/java')
-rw-r--r-- | node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/LoadBalancerProvisionerTest.java | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/LoadBalancerProvisionerTest.java b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/LoadBalancerProvisionerTest.java index eb3bdff484d..bdc3bdfd816 100644 --- a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/LoadBalancerProvisionerTest.java +++ b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/LoadBalancerProvisionerTest.java @@ -11,6 +11,7 @@ import com.yahoo.config.provision.HostSpec; import com.yahoo.config.provision.NodeResources; import com.yahoo.config.provision.NodeType; import com.yahoo.config.provision.exception.LoadBalancerServiceException; +import com.yahoo.transaction.NestedTransaction; import com.yahoo.vespa.flags.InMemoryFlagSource; import com.yahoo.vespa.hosted.provision.Node; import com.yahoo.vespa.hosted.provision.NodeList; @@ -175,7 +176,9 @@ public class LoadBalancerProvisionerTest { // Application is removed, nodes are deleted and load balancer is deactivated tester.remove(app1); - tester.nodeRepository().database().removeNodes(tester.nodeRepository().nodes().list().nodeType(NodeType.tenant).asList()); + NestedTransaction transaction = new NestedTransaction(); + tester.nodeRepository().database().removeNodes(tester.nodeRepository().nodes().list().nodeType(NodeType.tenant).asList(), transaction); + transaction.commit(); assertTrue("Nodes are deleted", tester.nodeRepository().nodes().list().nodeType(NodeType.tenant).isEmpty()); assertSame("Load balancer is deactivated", LoadBalancer.State.inactive, lb.get().state()); |