Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MLH-36 | Integrating OTEL with logback #3933

Merged
merged 26 commits into from
Jan 29, 2025
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
48ac86f
ATLAS-4917: replace use of log4j with logback
mneethiraj Nov 12, 2024
fa736fe
use log4j after logback
aarshi0301 Dec 24, 2024
7fabb7f
remove
aarshi0301 Dec 24, 2024
873adc1
OTEL integration after log back
aarshi0301 Dec 26, 2024
a6a87f0
Capture audit configurations as attributes
aarshi0301 Dec 31, 2024
5b383f0
remove clear
aarshi0301 Jan 2, 2025
3c1c672
Add resource attributes
aarshi0301 Jan 7, 2025
cd6b75c
fix filename
aarshi0301 Jan 9, 2025
fc5d98a
add mdc filter
aarshi0301 Jan 9, 2025
bbd59a3
add MDC filter for logging
aarshi0301 Jan 9, 2025
d189960
update logback file
aarshi0301 Jan 9, 2025
a0c0876
add logback for ranger
aarshi0301 Jan 9, 2025
197ec5e
record elapsed time as MDC attribute
aarshi0301 Jan 9, 2025
b548633
enable log4jaudit destination
aarshi0301 Jan 20, 2025
b5ae633
remove MDC attribute
aarshi0301 Jan 21, 2025
2f55038
fix: change loginterceptor level to basic to only print status log
sumandas0 Jan 21, 2025
3722974
fix: change logging level to NONE
sumandas0 Jan 21, 2025
6a1198e
Merge branch 'master' into otel3
aarshi0301 Jan 22, 2025
2496aa5
Merge branch 'mlh-36-fix-logging' into otel3
aarshi0301 Jan 22, 2025
46b3822
Remove irrelevant files
aarshi0301 Jan 23, 2025
c58d739
undo pom upstream changes
aarshi0301 Jan 23, 2025
d032780
Merge branch 'master' into otel3
aarshi0301 Jan 28, 2025
cd9e34c
add ranger indexes
aarshi0301 Jan 28, 2025
0111fbd
remove trace id
aarshi0301 Jan 28, 2025
89d031e
fix variable name
aarshi0301 Jan 28, 2025
1f04397
optimise casting
aarshi0301 Jan 28, 2025
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 1 addition & 2 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -56,8 +56,7 @@ COPY atlas-hub/repair_index.py /opt/apache-atlas/bin/
RUN chmod +x /opt/apache-atlas/bin/repair_index.py

COPY atlas-hub/atlas_start.py.patch atlas-hub/atlas_config.py.patch /opt/apache-atlas/bin/
COPY atlas-hub/pre-conf/atlas-log4j.xml /opt/apache-atlas/conf/
COPY atlas-hub/pre-conf/atlas-log4j2.xml /opt/apache-atlas/conf/
COPY atlas-hub/pre-conf/atlas-logback.xml /opt/apache-atlas/conf/
COPY atlas-hub/pre-conf/atlas-auth/ /opt/apache-atlas/conf/

RUN curl https://repo1.maven.org/maven2/org/jolokia/jolokia-jvm/1.6.2/jolokia-jvm-1.6.2-agent.jar -o /opt/apache-atlas/libext/jolokia-jvm-agent.jar
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,26 +17,22 @@
~ limitations under the License.
-->

<!DOCTYPE log4j:configuration SYSTEM "log4j.dtd">
<configuration>
<appender name="console" class="ch.qos.logback.core.ConsoleAppender">
<param name="Target" value="System.out"/>
<encoder>
<pattern>%date [%thread] %level{5} [%file:%line] %msg%n</pattern>
</encoder>
<filter class="ch.qos.logback.classic.filter.ThresholdFilter">
<level>INFO</level>
</filter>
</appender>

<log4j:configuration xmlns:log4j="http://jakarta.apache.org/log4j/">
<appender name="FILE" class="org.apache.log4j.RollingFileAppender">
<param name="File" value="/var/log/atlas/classification-updater.log"/>
<param name="Append" value="true"/>
<param name="maxFileSize" value="100MB" />
<param name="maxBackupIndex" value="20" />
<layout class="org.apache.log4j.PatternLayout">
<param name="ConversionPattern" value="%d %-5p - [%t:%x] ~ %m (%C{1}:%L)%n"/>
</layout>
</appender>
<logger name="org.apache.atlas" additivity="false" level="info">
<appender-ref ref="console"/>
</logger>

<logger name="org.apache.atlas.tools.BulkFetchAndUpdate" additivity="false">
<level value="info"/>
<appender-ref ref="FILE"/>
</logger>

<root>
<priority value="warn"/>
<appender-ref ref="FILE"/>
</root>
</log4j:configuration>
<root level="DEBUG">
<appender-ref ref="console"/>
</root>
</configuration>
4 changes: 2 additions & 2 deletions addons/falcon-bridge/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -267,8 +267,8 @@
<value>application.log</value>
</systemProperty>
<systemProperty>
<name>log4j.configuration</name>
<value>file:///${project.build.directory}/test-classes/atlas-log4j.xml</value>
<name>logback.configurationFile</name>
<value>file:///${project.build.directory}/test-classes/atlas-logback.xml</value>
</systemProperty>
<systemProperty>
<name>atlas.graphdb.backend</name>
Expand Down
137 changes: 0 additions & 137 deletions addons/falcon-bridge/src/test/resources/atlas-log4j.xml

This file was deleted.

132 changes: 132 additions & 0 deletions addons/falcon-bridge/src/test/resources/atlas-logback.xml
Original file line number Diff line number Diff line change
@@ -0,0 +1,132 @@
<?xml version="1.0" encoding="UTF-8" ?>
<!--
~ Licensed to the Apache Software Foundation (ASF) under one
~ or more contributor license agreements. See the NOTICE file
~ distributed with this work for additional information
~ regarding copyright ownership. The ASF licenses this file
~ to you under the Apache License, Version 2.0 (the
~ "License"); you may not use this file except in compliance
~ with the License. You may obtain a copy of the License at
~
~ http://www.apache.org/licenses/LICENSE-2.0
~
~ Unless required by applicable law or agreed to in writing, software
~ distributed under the License is distributed on an "AS IS" BASIS,
~ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
~ See the License for the specific language governing permissions and
~ limitations under the License.
-->

<configuration>
<appender name="console" class="ch.qos.logback.core.ConsoleAppender">
<param name="Target" value="System.out"/>
<encoder>
<pattern>%date [%thread] %level{5} [%file:%line] %msg%n</pattern>
</encoder>
<filter class="ch.qos.logback.classic.filter.ThresholdFilter">
<level>INFO</level>
</filter>
</appender>

<appender name="FILE" class="ch.qos.logback.core.rolling.RollingFileAppender">
<file>${atlas.log.dir}/${atlas.log.file}</file>
<append>true</append>
<encoder>
<pattern>%date [%thread] %level{5} [%file:%line] %msg%n</pattern>
</encoder>
<rollingPolicy class="ch.qos.logback.core.rolling.TimeBasedRollingPolicy">
<fileNamePattern>${atlas.log.dir}/${atlas.log.file}-%d</fileNamePattern>
<maxHistory>20</maxHistory>
<cleanHistoryOnStart>true</cleanHistoryOnStart>
</rollingPolicy>
</appender>

<appender name="AUDIT" class="ch.qos.logback.core.rolling.RollingFileAppender">
<file>${atlas.log.dir}/audit.log</file>
<append>true</append>
<encoder>
<pattern>%date [%thread] %level{5} [%file:%line] %msg%n</pattern>
</encoder>
<rollingPolicy class="ch.qos.logback.core.rolling.TimeBasedRollingPolicy">
<fileNamePattern>${atlas.log.dir}/audit-%d.log</fileNamePattern>
<maxHistory>20</maxHistory>
<cleanHistoryOnStart>false</cleanHistoryOnStart>
</rollingPolicy>
</appender>

<appender name="METRICS" class="ch.qos.logback.core.rolling.RollingFileAppender">
<file>${atlas.log.dir}/metrics.log</file>
<append>true</append>
<encoder>
<pattern>%date [%thread] %level{5} [%file:%line] %msg%n</pattern>
</encoder>
<rollingPolicy class="ch.qos.logback.core.rolling.TimeBasedRollingPolicy">
<fileNamePattern>${atlas.log.dir}/metrics-%d.log</fileNamePattern>
<maxHistory>20</maxHistory>
<cleanHistoryOnStart>false</cleanHistoryOnStart>
</rollingPolicy>
</appender>

<appender name="FAILED" class="ch.qos.logback.core.rolling.RollingFileAppender">
<file>${atlas.log.dir}/failed.log</file>
<append>true</append>
<encoder>
<pattern>%date [%thread] %level{5} [%file:%line] %msg%n</pattern>
</encoder>
<rollingPolicy class="ch.qos.logback.core.rolling.TimeBasedRollingPolicy">
<fileNamePattern>${atlas.log.dir}/failed-%d.log</fileNamePattern>
<maxHistory>20</maxHistory>
<cleanHistoryOnStart>false</cleanHistoryOnStart>
</rollingPolicy>
</appender>

<logger name="org.apache.atlas" additivity="false" level="info">
<appender-ref ref="FILE"/>
</logger>

<logger name="org.apache.atlas.impala.ImpalaLineageTool" additivity="false" level="debug">
<appender-ref ref="FILE"/>
</logger>

<logger name="org.apache.atlas.impala.hook.ImpalaLineageHook" additivity="false" level="debug">
<appender-ref ref="FILE"/>
</logger>

<logger name="org.janusgraph" additivity="false" level="warn">
<appender-ref ref="FILE"/>
</logger>

<logger name="org.springframework" additivity="false" level="warn">
<appender-ref ref="console"/>
</logger>

<logger name="org.eclipse" additivity="false" level="warn">
<appender-ref ref="console"/>
</logger>

<logger name="com.sun.jersey" additivity="false" level="warn">
<appender-ref ref="console"/>
</logger>


<!-- to avoid logs - The configuration log.flush.interval.messages = 1 was supplied but isn't a known config -->
<logger name="org.apache.kafka.common.config.AbstractConfig" additivity="false" level="error">
<appender-ref ref="FILE"/>
</logger>

<logger name="METRICS" additivity="false" level="debug">
<appender-ref ref="METRICS"/>
</logger>

<logger name="FAILED" additivity="false" level="info">
<appender-ref ref="FAILED"/>
</logger>

<logger name="AUDIT" additivity="false" level="info">
<appender-ref ref="AUDIT"/>
</logger>

<root level="warn">
<appender-ref ref="FILE"/>
</root>
</configuration>
9 changes: 7 additions & 2 deletions addons/hbase-bridge/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -339,6 +339,11 @@
<artifactId>jsr311-api</artifactId>
<version>${jsr.version}</version>
</artifactItem>
<artifactItem>
<groupId>ch.qos.logback</groupId>
<artifactId>logback-classic</artifactId>
<version>${logback.version}</version>
</artifactItem>
</artifactItems>
</configuration>
</execution>
Expand Down Expand Up @@ -417,8 +422,8 @@
<value>application.log</value>
</systemProperty>
<systemProperty>
<name>log4j.configuration</name>
<value>file:///${project.build.directory}/test-classes/atlas-log4j.xml</value>
<name>logback.configurationFile</name>
<value>file:///${project.build.directory}/test-classes/atlas-logback.xml</value>
</systemProperty>
<systemProperty>
<name>atlas.graphdb.backend</name>
Expand Down
Loading
Loading