summaryrefslogtreecommitdiffstats
path: root/controller-server
diff options
context:
space:
mode:
authorMartin Polden <mpolden@mpolden.no>2021-06-15 15:57:31 +0200
committerMartin Polden <mpolden@mpolden.no>2021-06-15 15:57:31 +0200
commite33c4ec399ddcfa7bcea65dcab53733a4e27aaf0 (patch)
treece6cd118178d23af641f096b39f1796db3ec221a /controller-server
parent79ef7e0298e10f007646f19f6bcabe6cd03932aa (diff)
Improve failure reason for suspension timeout
Diffstat (limited to 'controller-server')
-rw-r--r--controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/InternalStepRunner.java5
1 files changed, 4 insertions, 1 deletions
diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/InternalStepRunner.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/InternalStepRunner.java
index c7270b6c426..10f96ff13cd 100644
--- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/InternalStepRunner.java
+++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/InternalStepRunner.java
@@ -340,7 +340,10 @@ public class InternalStepRunner implements StepRunner {
.map(since -> since.isBefore(controller.clock().instant().minus(timeouts.noNodesDown())))
.orElse(false)) {
if (summary.needPlatformUpgrade() > 0 || summary.needReboot() > 0 || summary.needRestart() > 0)
- failureReason = "No nodes allowed to suspend to progress installation for " + timeouts.noNodesDown().toMinutes() + " minutes.";
+ failureReason = "Timed out after waiting " + timeouts.noNodesDown().toMinutes() + " minutes for " +
+ "nodes to suspend. This is normal if the cluster is excessively busy. " +
+ "Nodes will continue to attempt suspension to progress installation independently of " +
+ "this run.";
else
failureReason = "Nodes not able to start with new application package.";
}