summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorArne H Juul <arnej27959@users.noreply.github.com>2021-10-14 17:44:52 +0200
committerGitHub <noreply@github.com>2021-10-14 17:44:52 +0200
commitffcc28f2b9180ad8774c67fddbebc81b41f2fc6c (patch)
treec5ac7f737b47dcfd9b8cc1503f014031b8b2cc95
parent3d435694826f392e674480415faddf90c821e305 (diff)
parent057fd68a4ec3f0383db50ba429515fc376092cb0 (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.java5
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);
}