Merge pull request #14317 from vespa-engine/bratseth/only-log-when-we-redeploy

Only log when we really redeploy

1 files changed, 1 insertions, 2 deletions

diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/ApplicationMaintainer.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/ApplicationMaintainer.java
index 9980335bab0..0a94764d948 100644
--- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/ApplicationMaintainer.java
+++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/ApplicationMaintainer.java
@@ -64,8 +64,6 @@ public abstract class ApplicationMaintainer extends NodeRepositoryMaintainer {
      */
     protected void deploy(ApplicationId application) {
         if (pendingDeployments.addIfAbsent(application)) { // Avoid queuing multiple deployments for same application
-            log.log(Level.INFO, application + " will be deployed, last deploy time " +
-                                   getLastDeployTime(application));
             deploymentExecutor.execute(() -> deployWithLock(application));
         }
     }
@@ -84,6 +82,7 @@ public abstract class ApplicationMaintainer extends NodeRepositoryMaintainer {
         try (MaintenanceDeployment deployment = new MaintenanceDeployment(application, deployer, metric, nodeRepository())) {
             if ( ! deployment.isValid()) return false; // this will be done at another config server
             if ( ! canDeployNow(application)) return false; // redeployment is no longer needed
+            log.log(Level.INFO, application + " will be deployed, last deploy time " + getLastDeployTime(application));
             return deployment.activate();
         } finally {
             pendingDeployments.remove(application);