commit 8e0e69b2703714d66bab9f3a4488b544d4e3cf89
author Edwin Kempin <ekempin@google.com> Thu Oct 15 09:08:02 2020 +0200
committer Edwin Kempin <ekempin@google.com> Thu Oct 15 09:09:42 2020 +0200
tree 8f8f2d058dae243d1f7499cb4e9d948e4f3e38b1
parent 63b86f4f081e853f220c7f78426513b634d93254

CodeOwnerConfigValidator: Log cause if validation fails

It was the assumption that this is not needed since the thrown
StorageException should be logged, but this seems to be not always the
case.

Signed-off-by: Edwin Kempin <ekempin@google.com>
Change-Id: I71047cd079f13486ad9c7048561e72a66825b827

java/com/google/gerrit/plugins/codeowners/validation/CodeOwnerConfigValidator.java

diff --git a/java/com/google/gerrit/plugins/codeowners/validation/CodeOwnerConfigValidator.java b/java/com/google/gerrit/plugins/codeowners/validation/CodeOwnerConfigValidator.java
index f0023ff..86a206b 100644
--- a/java/com/google/gerrit/plugins/codeowners/validation/CodeOwnerConfigValidator.java
+++ b/java/com/google/gerrit/plugins/codeowners/validation/CodeOwnerConfigValidator.java
@@ -319,7 +319,7 @@
               "failed to validate code owner config files in revision %s"
                   + " (project = %s, branch = %s)",
               revCommit.getName(), branchNameKey.project(), branchNameKey.branch());
-      logger.atSevere().log(errorMessage);
+      logger.atSevere().withCause(e).log(errorMessage);
       throw new StorageException(errorMessage, e);
     }
   }