Merge pull request #2332 from yahoo/freva/additional-node-agent-logging

Log containerState and isFrozen transitions

1 files changed, 5 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 556c8c9f307..2ff10560fc1 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
@@ -120,6 +120,7 @@ public class NodeAgentImpl implements NodeAgent {
                         lastCpuMetric = new CpuUsageReporter(container.created);
                     }
                     containerState = RUNNING_HOWEVER_RESUME_SCRIPT_NOT_RUN;
+                    logger.info("Container is already running, setting containerState to " + containerState);
                 });
     }
 
@@ -217,9 +218,9 @@ public class NodeAgentImpl implements NodeAgent {
         experimentalWriteFile(nodeSpec);
 
         addDebugMessage("Starting optional node program resume command");
-        logger.info("Starting optional node program resume command");
         dockerOperations.resumeNode(containerName);
         containerState = RUNNING;
+        logger.info("Resume command successfully executed, new containerState is " + containerState);
     }
 
     private void updateNodeRepoAndMarkNodeAsReady(ContainerNodeSpec nodeSpec) {
@@ -269,6 +270,7 @@ public class NodeAgentImpl implements NodeAgent {
             addDebugMessage("startContainerIfNeeded: containerState " + containerState + " -> " +
                             RUNNING_HOWEVER_RESUME_SCRIPT_NOT_RUN);
             containerState = RUNNING_HOWEVER_RESUME_SCRIPT_NOT_RUN;
+            logger.info("Container successfully started, new containerState is " + containerState);
         }
     }
 
@@ -347,6 +349,7 @@ public class NodeAgentImpl implements NodeAgent {
             dockerOperations.removeContainer(existingContainer.get());
             metricReceiver.unsetMetricsForContainer(hostname);
             containerState = ABSENT;
+            logger.info("Container successfully removed, new containerState is " + containerState);
             return Optional.empty();
         }
         return existingContainer;
@@ -395,6 +398,7 @@ public class NodeAgentImpl implements NodeAgent {
 
             if (isFrozen != wantFrozen) {
                 isFrozen = wantFrozen;
+                logger.info("Updated NodeAgent's frozen state, new value: isFrozen: " + isFrozen);
             }
             isFrozenCopy = isFrozen;
         }