diff options
author | Harald Musum <musum@oath.com> | 2018-09-21 11:42:47 +0200 |
---|---|---|
committer | Harald Musum <musum@oath.com> | 2018-09-21 11:42:47 +0200 |
commit | 8e209f0d3bb00f806fb44a1e9d416d70000b49e0 (patch) | |
tree | 2549f5a133f08d2a3915bf99ee03656eff1fdf40 /config-model/src/test/java/com/yahoo/config | |
parent | 8d1b566b276f4f2f6abc271c1724cda0ca634c1e (diff) |
Fix missing wiring and enable test
Diffstat (limited to 'config-model/src/test/java/com/yahoo/config')
-rw-r--r-- | config-model/src/test/java/com/yahoo/config/model/provision/ModelProvisioningTest.java | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/config-model/src/test/java/com/yahoo/config/model/provision/ModelProvisioningTest.java b/config-model/src/test/java/com/yahoo/config/model/provision/ModelProvisioningTest.java index dd28d45ef4d..32a881633b8 100644 --- a/config-model/src/test/java/com/yahoo/config/model/provision/ModelProvisioningTest.java +++ b/config-model/src/test/java/com/yahoo/config/model/provision/ModelProvisioningTest.java @@ -48,6 +48,7 @@ import java.util.stream.Collectors; import static com.yahoo.config.model.test.TestUtil.joinLines; import static com.yahoo.vespa.defaults.Defaults.getDefaults; +import static org.hamcrest.CoreMatchers.both; import static org.hamcrest.CoreMatchers.is; import static org.hamcrest.collection.IsIn.isIn; import static org.hamcrest.core.Every.everyItem; @@ -947,11 +948,11 @@ public class ModelProvisioningTest { " <nodes count='1'/>" + " </container>" + "</services>"; - testContainerOnLogserverHost(services); + boolean useDedicatedNodeForLogserver = false; + testContainerOnLogserverHost(services, useDedicatedNodeForLogserver); } @Test - @Ignore // Ignore until we create container on logserver implicitly public void testImplicitLogserverContainer() { String services = "<?xml version='1.0' encoding='utf-8' ?>\n" + @@ -960,7 +961,8 @@ public class ModelProvisioningTest { " <nodes count='1'/>" + " </container>" + "</services>"; - testContainerOnLogserverHost(services); + boolean useDedicatedNodeForLogserver = true; + testContainerOnLogserverHost(services, useDedicatedNodeForLogserver); } @Test @@ -1751,9 +1753,10 @@ public class ModelProvisioningTest { // Tests that a container is allocated on logserver host and that // it is able to get config - private void testContainerOnLogserverHost(String services) { + private void testContainerOnLogserverHost(String services, boolean useDedicatedNodeForLogserver) { int numberOfHosts = 2; VespaModelTester tester = new VespaModelTester(); + tester.useDedicatedNodeForLogserver(useDedicatedNodeForLogserver); tester.addHosts(numberOfHosts); Zone zone = new Zone(SystemName.cd, Environment.prod, RegionName.defaultName()); |