summaryrefslogtreecommitdiffstats
path: root/node-admin
diff options
context:
space:
mode:
authorHåkon Hallingstad <hakon@verizonmedia.com>2019-11-14 15:18:56 +0100
committerHåkon Hallingstad <hakon@verizonmedia.com>2019-11-14 15:18:56 +0100
commite487aa3358eec6b0db471108e9b0a6d1ea6194e3 (patch)
treedbd3394f95b745d2a54bc65e73336f400dc4b76f /node-admin
parent03d90c743ae83cfea09be55cb7f1787aa8c8453b (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.java7
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