This is an automated email from the ASF dual-hosted git repository.

dimas pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/polaris.git


The following commit(s) were added to refs/heads/main by this push:
     new d3dbe0c3b Fix deprecated Quarkus log properties (#2216)
d3dbe0c3b is described below

commit d3dbe0c3b719a253b48ac6237697379b134bc9ba
Author: Christopher Lambert <xn...@gmx.de>
AuthorDate: Fri Aug 1 15:18:42 2025 +0200

    Fix deprecated Quarkus log properties (#2216)
    
    see 
https://github.com/quarkusio/quarkus/wiki/Migration-Guide-3.19#other-changes-gear-white_check_mark
    
    CI Quarkus tests were logging this repeatedly:
    ```
    The "quarkus.log.file.json" config property is deprecated and should not be 
used anymore.
    The "quarkus.log.console.json" config property is deprecated and should not 
be used anymore.
    The "quarkus.log.file.json" config property is deprecated and should not be 
used anymore.
    The "quarkus.log.console.json" config property is deprecated and should not 
be used anymore.
    ```
---
 helm/polaris/templates/configmap.yaml                      | 4 ++--
 helm/polaris/tests/configmap_test.yaml                     | 4 ++--
 runtime/defaults/src/main/resources/application.properties | 4 ++--
 site/content/in-dev/unreleased/telemetry.md                | 2 +-
 4 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/helm/polaris/templates/configmap.yaml 
b/helm/polaris/templates/configmap.yaml
index 8d4eef024..3e120765a 100644
--- a/helm/polaris/templates/configmap.yaml
+++ b/helm/polaris/templates/configmap.yaml
@@ -136,7 +136,7 @@ data:
     {{- $_ = set $map "quarkus.log.console.enable" "true" -}}
     {{- $_ = set $map "quarkus.log.console.level" 
.Values.logging.console.threshold -}}
     {{- if .Values.logging.console.json -}}
-    {{- $_ = set $map "quarkus.log.console.json" "true" -}}
+    {{- $_ = set $map "quarkus.log.console.json.enabled" "true" -}}
     {{- else -}}
     {{- $_ = set $map "quarkus.log.console.format" 
.Values.logging.console.format -}}
     {{- end -}}
@@ -153,7 +153,7 @@ data:
     {{- $_ = set $map "quarkus.log.file.rotation.file-suffix" 
.Values.logging.file.rotation.fileSuffix -}}
     {{- end -}}
     {{- if .Values.logging.file.json -}}
-    {{- $_ = set $map "quarkus.log.file.json" "true" -}}
+    {{- $_ = set $map "quarkus.log.file.json.enabled" "true" -}}
     {{- else -}}
     {{- $_ = set $map "quarkus.log.file.format" .Values.logging.file.format -}}
     {{- end -}}
diff --git a/helm/polaris/tests/configmap_test.yaml 
b/helm/polaris/tests/configmap_test.yaml
index 0c99c9672..eb6177076 100644
--- a/helm/polaris/tests/configmap_test.yaml
+++ b/helm/polaris/tests/configmap_test.yaml
@@ -273,8 +273,8 @@ tests:
     asserts:
       - matchRegex: { path: 'data["application.properties"]', pattern: 
"quarkus.log.file.enable=true" }
       - matchRegex: { path: 'data["application.properties"]', pattern: 
"quarkus.log.console.enable=true" }
-      - matchRegex: { path: 'data["application.properties"]', pattern: 
"quarkus.log.file.json=true" }
-      - matchRegex: { path: 'data["application.properties"]', pattern: 
"quarkus.log.console.json=true" }
+      - matchRegex: { path: 'data["application.properties"]', pattern: 
"quarkus.log.file.json.enabled=true" }
+      - matchRegex: { path: 'data["application.properties"]', pattern: 
"quarkus.log.console.json.enabled=true" }
 
   - it: should include logging categories
     set:
diff --git a/runtime/defaults/src/main/resources/application.properties 
b/runtime/defaults/src/main/resources/application.properties
index 7e9fbadac..11f9cf9ab 100644
--- a/runtime/defaults/src/main/resources/application.properties
+++ b/runtime/defaults/src/main/resources/application.properties
@@ -58,11 +58,11 @@ quarkus.http.test-port=0
 quarkus.log.level=INFO
 quarkus.log.console.enable=true
 quarkus.log.console.level=ALL
-quarkus.log.console.json=false
+quarkus.log.console.json.enabled=false
 quarkus.log.console.format=%d{yyyy-MM-dd HH:mm:ss,SSS} %-5p [%c{3.}] 
[%X{requestId},%X{realmId}] [%X{traceId},%X{parentId},%X{spanId},%X{sampled}] 
(%t) %s%e%n
 quarkus.log.file.enable=true
 quarkus.log.file.level=ALL
-quarkus.log.file.json=false
+quarkus.log.file.json.enabled=false
 quarkus.log.file.format=%d{yyyy-MM-dd HH:mm:ss,SSS} %-5p [%c{3.}] 
[%X{requestId},%X{realmId}] [%X{traceId},%X{parentId},%X{spanId},%X{sampled}] 
(%t) %s%e%n
 quarkus.log.file.path=./logs/polaris.log
 quarkus.log.file.rotation.file-suffix=.yyyy-MM-dd.gz
diff --git a/site/content/in-dev/unreleased/telemetry.md 
b/site/content/in-dev/unreleased/telemetry.md
index 5921586d5..9e867408d 100644
--- a/site/content/in-dev/unreleased/telemetry.md
+++ b/site/content/in-dev/unreleased/telemetry.md
@@ -137,7 +137,7 @@ By default, logs are written to the console and to a file 
located in the `./logs
 file is rotated daily and compressed. The maximum size of the log file is 
10MB, and the maximum
 number of backup files is 14.
 
-JSON logging can be enabled by setting the `quarkus.log.console.json` and 
`quarkus.log.file.json`
+JSON logging can be enabled by setting the `quarkus.log.console.json.enabled` 
and `quarkus.log.file.json.enabled`
 properties to `true`. By default, JSON logging is disabled.
 
 The log level can be set for the entire application or for specific packages. 
The default log level

Reply via email to