Improved logging for NodeProvisioner.

sekhar09 · Jun 18, 2019 · ee9edf8 · ee9edf8
1 parent bc40ab3
commit ee9edf8
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/core/src/main/java/hudson/slaves/NodeProvisioner.java b/core/src/main/java/hudson/slaves/NodeProvisioner.java
@@ -174,9 +174,12 @@ public void suggestReviewNow() {
             lastSuggestedReview = System.currentTimeMillis();
             Computer.threadPoolForRemoting.submit(new Runnable() {
                 public void run() {
+                    LOGGER.fine(() -> "running suggested review for " + label);
                     update();
                 }
             });
+        } else {
+            LOGGER.fine(() -> "ignoring suggested review for " + label);
         }
     }
 
@@ -188,6 +191,7 @@ public void run() {
      * instance of this provisioner is running at a time) and then a lock on {@link Queue#lock}
      */
     private void update() {
+        long start = LOGGER.isLoggable(Level.FINER) ? System.nanoTime() : 0;
         provisioningLock.lock();
         try {
             lastSuggestedReview = System.currentTimeMillis();
@@ -327,6 +331,9 @@ public void run() {
         } finally {
             provisioningLock.unlock();
         }
+        if (LOGGER.isLoggable(Level.FINER)) {
+            LOGGER.finer(() -> "ran update on " + label + " in " + (System.nanoTime() - start) / 1_000_000 + "ms");
+        }
     }