From 08df5af4be85b9085dc68ca9c099bb8470f0d179 Mon Sep 17 00:00:00 2001 From: Håkon Hallingstad Date: Sun, 18 Jun 2017 02:29:25 +0200 Subject: Log a bit more in request executor --- .../hosted/node/admin/util/ConfigServerHttpRequestExecutor.java | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) (limited to 'node-admin') diff --git a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/util/ConfigServerHttpRequestExecutor.java b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/util/ConfigServerHttpRequestExecutor.java index 3358397a724..4434213989f 100644 --- a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/util/ConfigServerHttpRequestExecutor.java +++ b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/util/ConfigServerHttpRequestExecutor.java @@ -84,7 +84,12 @@ public class ConfigServerHttpRequestExecutor { response = client.execute(requestFactory.createRequest(configServer)); } catch (Exception e) { // Failure to communicate with a config server is not abnormal, as they are - // upgraded at the same time as Docker hosts. But it is abnormal if all of them are. + // upgraded at the same time as Docker hosts. + if (e.getMessage().indexOf("(Connection refused)") > 0) { + NODE_ADMIN_LOGGER.info("Connection refused to " + configServer + " (upgrading?), will try next"); + } else { + NODE_ADMIN_LOGGER.warning("Failed to communicate with " + configServer + ", will try next: " + e.getMessage()); + } lastException = e; continue; } -- cgit v1.2.3