summaryrefslogtreecommitdiffstats
path: root/node-admin
diff options
context:
space:
mode:
authorValerij Fredriksen <valerijf@oath.com>2018-08-09 15:41:43 +0200
committerValerij Fredriksen <valerijf@oath.com>2018-08-09 15:41:43 +0200
commitdbd0acddebf68e728569dc9d5cd2a174ca4fbdaa (patch)
tree2029fb0e6233dbdad36587f54225a286620440b0 /node-admin
parent65e8cce84c50607cef70e52cf542357ac6328158 (diff)
Add parent host by default in DockerTester
Diffstat (limited to 'node-admin')
-rw-r--r--node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/integrationTests/DockerTester.java13
-rw-r--r--node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/integrationTests/MultiDockerTest.java2
2 files changed, 14 insertions, 1 deletions
diff --git a/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/integrationTests/DockerTester.java b/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/integrationTests/DockerTester.java
index e8236bc826e..6960ce3e3f1 100644
--- a/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/integrationTests/DockerTester.java
+++ b/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/integrationTests/DockerTester.java
@@ -2,6 +2,7 @@
package com.yahoo.vespa.hosted.node.admin.integrationTests;
import com.yahoo.concurrent.classlock.ClassLocking;
+import com.yahoo.config.provision.NodeType;
import com.yahoo.metrics.simple.MetricReceiver;
import com.yahoo.vespa.hosted.dockerapi.Docker;
import com.yahoo.vespa.hosted.dockerapi.metrics.MetricReceiverWrapper;
@@ -20,6 +21,7 @@ import com.yahoo.vespa.hosted.node.admin.component.Environment;
import com.yahoo.vespa.hosted.node.admin.task.util.network.IPAddressesImpl;
import com.yahoo.vespa.hosted.node.admin.util.InetAddressResolver;
import com.yahoo.vespa.hosted.node.admin.component.PathResolver;
+import com.yahoo.vespa.hosted.provision.Node;
import java.net.InetAddress;
import java.net.UnknownHostException;
@@ -70,6 +72,17 @@ public class DockerTester implements AutoCloseable {
.pathResolver(new PathResolver(PATH_TO_VESPA_HOME, Paths.get("/tmp"), Paths.get("/tmp")))
.cloud("mycloud")
.build();
+
+ NodeSpec hostSpec = new NodeSpec.Builder()
+ .hostname(DOCKER_HOST_HOSTNAME)
+ .state(Node.State.active)
+ .nodeType(NodeType.host)
+ .flavor("default")
+ .wantedRestartGeneration(1L)
+ .currentRestartGeneration(1L)
+ .build();
+ nodeRepositoryMock.updateNodeRepositoryNode(hostSpec);
+
Clock clock = Clock.systemUTC();
DockerOperations dockerOperations = new DockerOperationsImpl(dockerMock, environment, null, new IPAddressesImpl());
StorageMaintainerMock storageMaintainer = new StorageMaintainerMock(dockerOperations, null, environment, callOrderVerifier, clock);
diff --git a/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/integrationTests/MultiDockerTest.java b/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/integrationTests/MultiDockerTest.java
index db0b8d1ed6c..7d2faf27bc5 100644
--- a/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/integrationTests/MultiDockerTest.java
+++ b/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/integrationTests/MultiDockerTest.java
@@ -80,7 +80,7 @@ public class MultiDockerTest {
tester.addChildNodeRepositoryNode(nodeSpec);
// Wait for node admin to be notified with node repo state and the docker container has been started
- while (tester.nodeAdmin.getListOfHosts().size() != tester.nodeRepositoryMock.getNumberOfContainerSpecs()) {
+ while (tester.nodeAdmin.getListOfHosts().size() + 1 != tester.nodeRepositoryMock.getNumberOfContainerSpecs()) {
Thread.sleep(10);
}