diff options
author | Håkon Hallingstad <hakon@verizonmedia.com> | 2019-11-14 15:18:56 +0100 |
---|---|---|
committer | Håkon Hallingstad <hakon@verizonmedia.com> | 2019-11-14 15:18:56 +0100 |
commit | e487aa3358eec6b0db471108e9b0a6d1ea6194e3 (patch) | |
tree | dbd3394f95b745d2a54bc65e73336f400dc4b76f /node-admin | |
parent | 03d90c743ae83cfea09be55cb7f1787aa8c8453b (diff) |
Increase client-side node-admin ConfigServer timeouts from 15s to 30s
Diffstat (limited to 'node-admin')
-rw-r--r-- | node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/configserver/ConfigServerApiImpl.java | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/configserver/ConfigServerApiImpl.java b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/configserver/ConfigServerApiImpl.java index 3f6909b8ea8..167bbcc1302 100644 --- a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/configserver/ConfigServerApiImpl.java +++ b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/configserver/ConfigServerApiImpl.java @@ -202,9 +202,10 @@ public class ConfigServerApiImpl implements ConfigServerApi { cm.setMaxTotal(200); // Increase max total connections to 200, which should be enough // Have experienced hang in socket read, which may have been because of - // system defaults, therefore set explicit timeouts. Set arbitrarily to - // 15s > 10s used by Orchestrator lock timeout. - int timeoutMs = 15_000; + // system defaults, therefore set explicit timeouts. Even though the Orchestrator + // server-side timeout is 10s, a 409 has been observed to be returned after ~30s + // in a case possibly involving zk leader election. + int timeoutMs = 30_000; RequestConfig requestBuilder = RequestConfig.custom() .setConnectTimeout(timeoutMs) // establishment of connection .setConnectionRequestTimeout(timeoutMs) // connection from connection manager |