diff options
| author | Bjørn Christian Seime <bjorncs@vespa.ai> | 2023-10-17 16:52:06 +0200 |
|---|---|---|
| committer | Bjørn Christian Seime <bjorncs@vespa.ai> | 2023-10-17 16:52:11 +0200 |
| commit | 9aba09c8196e007ab61fcd90e9b132ae798cbcd5 (patch) | |
| tree | 3da053ec5405364eef66165ea6ed0f90d2df88b0 /node-admin/src | |
| parent | 8d12c495d1c76815545ab115a73d29a4afdd0a47 (diff) | |
Trigger service dump earlier in lifecycle flow
Diffstat (limited to 'node-admin/src')
| -rw-r--r-- | node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/nodeagent/NodeAgentImpl.java | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/nodeagent/NodeAgentImpl.java b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/nodeagent/NodeAgentImpl.java index c3ec9e91343..43dc3d72c46 100644 --- a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/nodeagent/NodeAgentImpl.java +++ b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/nodeagent/NodeAgentImpl.java @@ -518,6 +518,8 @@ public class NodeAgentImpl implements NodeAgent { container = Optional.of(updateContainerIfNeeded(context, container.get())); } + serviceDumper.processServiceDumpRequest(context); + startServicesIfNeeded(context); resumeNodeIfNeeded(context); if (healthChecker.isPresent()) { @@ -531,7 +533,6 @@ public class NodeAgentImpl implements NodeAgent { throw ConvergenceException.ofTransient("Refusing to resume until warm up period ends (" + (timeLeft.isNegative() ? "next tick" : "in " + timeLeft) + ")"); } - serviceDumper.processServiceDumpRequest(context); // Because it's more important to stop a bad release from rolling out in prod, // we put the resume call last. So if we fail after updating the node repo attributes |