commit a82b780235da61e79967549bca8fd7d249d66bda
author Edwin Kempin <ekempin@google.com> Thu Mar 04 13:33:12 2021 +0100
committer Edwin Kempin <ekempin@google.com> Thu Mar 04 13:34:27 2021 +0100
tree 3fee6377c7394e8cfd75b6038a481dac130159c6
parent 331e6bdcb05641e32909a59d4610f4f5fd28bf34

CodeOwnerApprovalCheck: Add debug logs for pure reverts

We should be able to see from a trace whether pure reverts are
configured to be exempted from requiring code owner approvals and
whether a change was found to be a pure revert.

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

java/com/google/gerrit/plugins/codeowners/backend/CodeOwnerApprovalCheck.java

diff --git a/java/com/google/gerrit/plugins/codeowners/backend/CodeOwnerApprovalCheck.java b/java/com/google/gerrit/plugins/codeowners/backend/CodeOwnerApprovalCheck.java
index 4c41510..b563d84 100644
--- a/java/com/google/gerrit/plugins/codeowners/backend/CodeOwnerApprovalCheck.java
+++ b/java/com/google/gerrit/plugins/codeowners/backend/CodeOwnerApprovalCheck.java
@@ -238,7 +238,11 @@
       CodeOwnersPluginConfigSnapshot codeOwnersConfig =
           codeOwnersPluginConfiguration.getProjectConfig(changeNotes.getProjectName());
 
-      if (codeOwnersConfig.arePureRevertsExempted() && isPureRevert(changeNotes)) {
+      boolean arePureRevertsExempted = codeOwnersConfig.arePureRevertsExempted();
+      logger.atFine().log("arePureRevertsExempted = %s", arePureRevertsExempted);
+      if (arePureRevertsExempted && isPureRevert(changeNotes)) {
+        logger.atFine().log(
+            "change is a pure revert and is exempted from requiring code owner approvals");
         return getAllPathsAsApproved(changeNotes, changeNotes.getCurrentPatchSet());
       }