summaryrefslogtreecommitdiffstats
path: root/config-model
diff options
context:
space:
mode:
authorjonmv <venstad@gmail.com>2022-11-10 10:02:58 +0100
committerjonmv <venstad@gmail.com>2022-11-10 10:02:58 +0100
commit9b25175f39de4b16de0993780861dbbfdfa195d6 (patch)
treee6f684bfeb11e136a25544d4156dfee28574b3fd /config-model
parentfb4b572a7e57a00854be50516caaff22deca7419 (diff)
Test for configservers and forward to super
Diffstat (limited to 'config-model')
-rw-r--r--config-model/src/main/java/com/yahoo/vespa/model/container/ApplicationContainerCluster.java4
-rw-r--r--config-model/src/test/java/com/yahoo/vespa/model/container/configserver/ConfigserverClusterTest.java1
2 files changed, 3 insertions, 2 deletions
diff --git a/config-model/src/main/java/com/yahoo/vespa/model/container/ApplicationContainerCluster.java b/config-model/src/main/java/com/yahoo/vespa/model/container/ApplicationContainerCluster.java
index 732b1048016..f01d88c3c23 100644
--- a/config-model/src/main/java/com/yahoo/vespa/model/container/ApplicationContainerCluster.java
+++ b/config-model/src/main/java/com/yahoo/vespa/model/container/ApplicationContainerCluster.java
@@ -313,10 +313,10 @@ public final class ApplicationContainerCluster extends ContainerCluster<Applicat
@Override
public void getConfig(CuratorConfig.Builder builder) {
- if (getParent() instanceof ConfigserverCluster) return; // Produces its own config
super.getConfig(builder);
+ if (getParent() instanceof ConfigserverCluster) return; // Produces its own config
- // Will be forced between 2x and 20x of ZookeeperServerConfig.tickTime(), which is currently 6s.
+ // Will be bounded by 2x and 20x ZookeeperServerConfig.tickTime(), which is currently 6s.
builder.zookeeperSessionTimeoutSeconds(zookeeperSessionTimeoutSeconds);
}
diff --git a/config-model/src/test/java/com/yahoo/vespa/model/container/configserver/ConfigserverClusterTest.java b/config-model/src/test/java/com/yahoo/vespa/model/container/configserver/ConfigserverClusterTest.java
index c71f3946937..72b7e7649fd 100644
--- a/config-model/src/test/java/com/yahoo/vespa/model/container/configserver/ConfigserverClusterTest.java
+++ b/config-model/src/test/java/com/yahoo/vespa/model/container/configserver/ConfigserverClusterTest.java
@@ -121,6 +121,7 @@ public class ConfigserverClusterTest {
assertEquals(1, config.server().size());
assertEquals("localhost", config.server().get(0).hostname());
assertEquals(2181, config.server().get(0).port());
+ assertEquals(120, config.zookeeperSessionTimeoutSeconds());
assertTrue(config.zookeeperLocalhostAffinity());
}