diff options
author | Arne H Juul <arnej27959@users.noreply.github.com> | 2021-10-14 17:44:52 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-10-14 17:44:52 +0200 |
commit | ffcc28f2b9180ad8774c67fddbebc81b41f2fc6c (patch) | |
tree | c5ac7f737b47dcfd9b8cc1503f014031b8b2cc95 | |
parent | 3d435694826f392e674480415faddf90c821e305 (diff) | |
parent | 057fd68a4ec3f0383db50ba429515fc376092cb0 (diff) |
Merge pull request #19557 from vespa-engine/arnej/remove-unneeded-overrides
no need to override config with the same values as the defaults
-rw-r--r-- | config-model/src/main/java/com/yahoo/vespa/model/ConfigSentinel.java | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/config-model/src/main/java/com/yahoo/vespa/model/ConfigSentinel.java b/config-model/src/main/java/com/yahoo/vespa/model/ConfigSentinel.java index 7f74a5651fb..2eb8a6e319b 100644 --- a/config-model/src/main/java/com/yahoo/vespa/model/ConfigSentinel.java +++ b/config-model/src/main/java/com/yahoo/vespa/model/ConfigSentinel.java @@ -85,10 +85,7 @@ public class ConfigSentinel extends AbstractService implements SentinelConfig.Pr private SentinelConfig.Connectivity.Builder getConnectivityConfig(boolean enable) { var builder = new SentinelConfig.Connectivity.Builder(); - if (enable) { - builder.minOkPercent(50); - builder.maxBadCount(1); - } else { + if (! enable) { builder.minOkPercent(0); builder.maxBadCount(Integer.MAX_VALUE); } |