commit 904b168155108aeb6b0e2f9707d237d46f5873f9
author David Pursehouse <dpursehouse@collab.net> Wed May 09 10:00:08 2018 +0900
committer David Pursehouse <dpursehouse@collab.net> Wed May 09 10:05:10 2018 +0900
tree e5ca6202a5d8c019de18845c1b8ec630e1c59e49
parent 4aa4eac2613ed8c4dd0ba4c61f5e08ecb21fc60a

Log a warning when a hook times out

Change-Id: I71bf3ec5f2c85470ff91b7afe80a82a2df0c42e3

src/main/java/com/googlesource/gerrit/plugins/hooks/HookTask.java

diff --git a/src/main/java/com/googlesource/gerrit/plugins/hooks/HookTask.java b/src/main/java/com/googlesource/gerrit/plugins/hooks/HookTask.java
index a0b05cf..edbcc01 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/hooks/HookTask.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/hooks/HookTask.java
@@ -113,6 +113,7 @@
     } catch (InterruptedException iex) {
       // InterruptedException - timeout or cancel
       args.metrics.timeout(name);
+      log.warn("hook[{}] timed out: {}", name, iex.getMessage());
     } catch (Throwable err) {
       args.metrics.error(name);
       log.error("Error running hook {}", hook.toAbsolutePath(), err);