Downgrade WorkflowFailedResponse Evennt log level to INFO

AlekseiLitkovetc · Jul 15, 2019 · 123fcab · 123fcab
1 parent dd1c46f
commit 123fcab
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/engine/src/main/scala/cromwell/engine/workflow/WorkflowManagerActor.scala b/engine/src/main/scala/cromwell/engine/workflow/WorkflowManagerActor.scala
@@ -179,8 +179,11 @@ class WorkflowManagerActor(params: WorkflowManagerActorParams)
     /*
      Responses from services
      */
+    /*
+     BA-5800: Reduce sentry logging, downgrade log level from error to info
+     */
     case Event(WorkflowFailedResponse(workflowId, inState, reasons), _) =>
-      log.error(s"$tag Workflow $workflowId failed (during $inState): ${expandFailureReasons(reasons)}")
+      log.info(s"$tag Workflow $workflowId failed (during $inState): ${expandFailureReasons(reasons)}")
       stay()
     /*
      Watched transitions

diff --git a/server/src/main/resources/logback.xml b/server/src/main/resources/logback.xml
@@ -77,10 +77,18 @@
         </else>
     </if>
 
+   <!-- Configure the Sentry appender, overriding the logging threshold to the WARN level -->
+    <appender name="Sentry" class="io.sentry.logback.SentryAppender">
+        <filter class="ch.qos.logback.classic.filter.ThresholdFilter">
+            <level>WARN</level>
+        </filter>
+    </appender>
+
     <root level="${LOG_LEVEL}">
         <appender-ref ref="STANDARD_APPENDER" />
         <appender-ref ref="PRETTY_APPENDER" />
         <appender-ref ref="FILEROLLER_APPENDER" />
+        <appender-ref ref="Sentry" />
     </root>
 
     <logger name="com.zaxxer.hikari" level="ERROR"/>