commit 9d5f75347182c330b405e33419080125598b40c9
author Edwin Kempin <ekempin@google.com> Tue Sep 21 16:57:54 2021 +0200
committer Edwin Kempin <ekempin@google.com> Tue Sep 21 16:59:59 2021 +0200
tree d1cf113fea7bd5e53e49f6bc367e7194ba3812ea
parent 6343a807a93952792b05802f6dbd9265c0de844e

ServiceUserClassifierImpl: Reduce log about cyclic group to fine level

Having cyclic groups is not a problem for Gerrit, and hence it's okay if
cyclic groups exist. We do not need a warning about this. Reduce the log
level to FINE to reduce noise in our logs.

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

java/com/google/gerrit/server/account/ServiceUserClassifierImpl.java

diff --git a/java/com/google/gerrit/server/account/ServiceUserClassifierImpl.java b/java/com/google/gerrit/server/account/ServiceUserClassifierImpl.java
index 27ac9f4..db030f9 100644
--- a/java/com/google/gerrit/server/account/ServiceUserClassifierImpl.java
+++ b/java/com/google/gerrit/server/account/ServiceUserClassifierImpl.java
@@ -76,7 +76,7 @@
               .get(toTraverse.remove(0))
               .orElseThrow(() -> new IllegalStateException("invalid subgroup"));
       if (seen.contains(currentGroup.getGroupUUID())) {
-        logger.atWarning().log(
+        logger.atFine().log(
             "Skipping %s because it's a cyclic subgroup", currentGroup.getGroupUUID());
         continue;
       }