diff options
author | jonmv <venstad@gmail.com> | 2022-11-10 10:02:58 +0100 |
---|---|---|
committer | jonmv <venstad@gmail.com> | 2022-11-10 10:02:58 +0100 |
commit | 9b25175f39de4b16de0993780861dbbfdfa195d6 (patch) | |
tree | e6f684bfeb11e136a25544d4156dfee28574b3fd /config-model | |
parent | fb4b572a7e57a00854be50516caaff22deca7419 (diff) |
Test for configservers and forward to super
Diffstat (limited to 'config-model')
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()); } |