Skip to content

Commit 75a9029

Browse files
authored
Remove SLF4J and rm warning from the Strimzi build (#12303)
Signed-off-by: Jakub Scholz <www@scholzj.com>
1 parent 15009d6 commit 75a9029

File tree

3 files changed

+38
-1
lines changed

3 files changed

+38
-1
lines changed

config-model-generator/build-config-models.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ get_kafka_versions
99

1010
# Always delete existing files so when kafka-versions changes we remove
1111
# models for unsupported versions
12-
rm ../cluster-operator/src/main/resources/kafka-*-config-model.json || true
12+
rm -f ../cluster-operator/src/main/resources/kafka-*-config-model.json || true
1313

1414
if [ "$1" = "build" ]
1515
then

config-model-generator/pom.xml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -70,6 +70,10 @@
7070
<groupId>com.fasterxml.jackson.core</groupId>
7171
<artifactId>jackson-databind</artifactId>
7272
</dependency>
73+
<dependency>
74+
<groupId>org.apache.logging.log4j</groupId>
75+
<artifactId>log4j-slf4j2-impl</artifactId>
76+
</dependency>
7377
</dependencies>
7478

7579
<profiles>
@@ -98,6 +102,8 @@
98102
<ignoredUnusedDeclaredDependencies>
99103
<!-- Mot used directly because of differences between Kafka versions. But needed in the classpath. -->
100104
<ignoredUnusedDeclaredDependency>org.apache.kafka:kafka-raft</ignoredUnusedDeclaredDependency>
105+
<!-- Needed for logging using the Kafka Clients (uses SLF4J) -->
106+
<ignoredUnusedDeclaredDependency>org.apache.logging.log4j:log4j-slf4j2-impl</ignoredUnusedDeclaredDependency>
101107
</ignoredUnusedDeclaredDependencies>
102108
</configuration>
103109
</execution>
@@ -154,6 +160,8 @@
154160
<ignoredUnusedDeclaredDependencies>
155161
<!-- Mot used directly because of differences between Kafka versions. But needed in the classpath. -->
156162
<ignoredUnusedDeclaredDependency>org.apache.kafka:kafka-raft</ignoredUnusedDeclaredDependency>
163+
<!-- Needed for logging using the Kafka Clients (uses SLF4J) -->
164+
<ignoredUnusedDeclaredDependency>org.apache.logging.log4j:log4j-slf4j2-impl</ignoredUnusedDeclaredDependency>
157165
</ignoredUnusedDeclaredDependencies>
158166
</configuration>
159167
</execution>

crd-generator/pom.xml

Lines changed: 29 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,14 @@
5454
<groupId>com.fasterxml.jackson.core</groupId>
5555
<artifactId>jackson-annotations</artifactId>
5656
</dependency>
57+
<dependency>
58+
<groupId>org.apache.logging.log4j</groupId>
59+
<artifactId>log4j-core</artifactId>
60+
</dependency>
61+
<dependency>
62+
<groupId>org.apache.logging.log4j</groupId>
63+
<artifactId>log4j-slf4j2-impl</artifactId>
64+
</dependency>
5765
<dependency>
5866
<groupId>org.junit.jupiter</groupId>
5967
<artifactId>junit-jupiter-api</artifactId>
@@ -68,6 +76,27 @@
6876

6977
<build>
7078
<plugins>
79+
<plugin>
80+
<groupId>org.apache.maven.plugins</groupId>
81+
<artifactId>maven-dependency-plugin</artifactId>
82+
<version>${maven.dependency.version}</version>
83+
<executions>
84+
<execution>
85+
<id>analyze</id>
86+
<goals>
87+
<goal>analyze-only</goal>
88+
</goals>
89+
<configuration>
90+
<failOnWarning>true</failOnWarning>
91+
<ignoredUnusedDeclaredDependencies>
92+
<!-- Needed for logging using the Kubernetes Client (uses SLF4J) -->
93+
<ignoredUnusedDeclaredDependency>org.apache.logging.log4j:log4j-core</ignoredUnusedDeclaredDependency>
94+
<ignoredUnusedDeclaredDependency>org.apache.logging.log4j:log4j-slf4j2-impl</ignoredUnusedDeclaredDependency>
95+
</ignoredUnusedDeclaredDependencies>
96+
</configuration>
97+
</execution>
98+
</executions>
99+
</plugin>
71100
<plugin>
72101
<groupId>org.apache.maven.plugins</groupId>
73102
<artifactId>maven-shade-plugin</artifactId>

0 commit comments

Comments
 (0)