Skip to content

Commit 4341dfc

Browse files
committed
Move SLF4J log level config from workflow to pom.xml
Removes SLF4J log level and Apache Jena logger configuration from GitHub Actions workflow and sets them as system properties in the Maven Surefire plugin configuration in the respective pom.xml files. This centralizes logging configuration within the project build files for consistency across environments.
1 parent 94e0deb commit 4341dfc

File tree

3 files changed

+11
-3
lines changed

3 files changed

+11
-3
lines changed

.github/workflows/pull_request_check.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -48,13 +48,13 @@ jobs:
4848
4949
- name: Creating native image (Mac)
5050
if: matrix.os == 'macos-13'
51-
run: mvn install -P native-image,native-test-config -Dos.platform=mac -Dmaven.wagon.httpconnectionManager.ttlSeconds=60 -Dorg.slf4j.simpleLogger.defaultLogLevel=info -Dorg.slf4j.simpleLogger.log.org.apache.jena=info
51+
run: mvn install -P native-image,native-test-config -Dos.platform=mac -Dmaven.wagon.httpconnectionManager.ttlSeconds=60
5252
env:
5353
TOKEN: ${{ secrets.GITHUB_TOKEN }}
5454

5555
- name: Creating native image (Linux)
5656
if: matrix.os == 'ubuntu-latest'
57-
run: mvn install -P native-image,native-test-config -Dos.platform=linux -Dmaven.wagon.httpconnectionManager.ttlSeconds=60 -Dorg.slf4j.simpleLogger.defaultLogLevel=info -Dorg.slf4j.simpleLogger.log.org.apache.jena=info
57+
run: mvn install -P native-image,native-test-config -Dos.platform=linux -Dmaven.wagon.httpconnectionManager.ttlSeconds=60
5858
env:
5959
TOKEN: ${{ secrets.GITHUB_TOKEN }}
6060

@@ -64,7 +64,7 @@ jobs:
6464

6565
- name: Creating native image (Win)
6666
if: matrix.os == 'windows-latest'
67-
run: mvn install -P native-image,native-test-config -D os.platform=win -D maven.wagon.httpconnectionManager.ttlSeconds=60 -Dorg.slf4j.simpleLogger.defaultLogLevel=info -Dorg.slf4j.simpleLogger.log.org.apache.jena=info
67+
run: mvn install -P native-image,native-test-config -D os.platform=win -D maven.wagon.httpconnectionManager.ttlSeconds=60
6868
env:
6969
TOKEN: ${{ secrets.GITHUB_TOKEN }}
7070

aspect-model-editor-runtime/pom.xml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -342,6 +342,10 @@
342342
<reuseForks>false</reuseForks>
343343
<argLine>--add-exports org.graalvm.nativeimage/org.graalvm.nativeimage.impl=ALL-UNNAMED</argLine>
344344
<argLine>-agentlib:native-image-agent=config-merge-dir=${native-config-path}</argLine>
345+
<systemPropertyVariables>
346+
<org.slf4j.simpleLogger.defaultLogLevel>info</org.slf4j.simpleLogger.defaultLogLevel>
347+
<org.slf4j.simpleLogger.log.org.apache.jena>info</org.slf4j.simpleLogger.log.org.apache.jena>
348+
</systemPropertyVariables>
345349
</configuration>
346350
</plugin>
347351
<plugin>

aspect-model-editor-service/pom.xml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -156,6 +156,10 @@
156156
<configuration>
157157
<skip>false</skip>
158158
<argLine>-agentlib:native-image-agent=config-output-dir=${project.build.directory}/native-config</argLine>
159+
<systemPropertyVariables>
160+
<org.slf4j.simpleLogger.defaultLogLevel>info</org.slf4j.simpleLogger.defaultLogLevel>
161+
<org.slf4j.simpleLogger.log.org.apache.jena>info</org.slf4j.simpleLogger.log.org.apache.jena>
162+
</systemPropertyVariables>
159163
</configuration>
160164
</plugin>
161165
</plugins>

0 commit comments

Comments
 (0)