From 02030d9cc80ed09224027faddb2879ad7a4f9067 Mon Sep 17 00:00:00 2001
From: Gabriel Mainberger <gabriel.mainberger@vshn.net>
Date: Fri, 23 Feb 2024 17:45:25 +0100
Subject: [PATCH] Remove the JDK parameter java.awt.headless=true

There is a a lot of evidence that the parameter is set to `java.awt.headless=true` in Quarkus by default.

* Quarkus Building native Images: https://quarkus.io/guides/building-native-image) see default value for `java.awt.headless`
* https://github.com/quarkusio/quarkus/issues/20565
* https://github.com/quarkusio/quarkus/pull/20850

Signed-off-by: Gabriel Mainberger <gabriel.mainberger@vshn.net>
---
 charts/keycloakx/README.md                                     | 3 ++-
 charts/keycloakx/ci/h2-values.yaml                             | 1 -
 .../examples/postgresql-kubeping/keycloak-server-values.yaml   | 1 -
 .../keycloakx/examples/postgresql/keycloak-server-values.yaml  | 1 -
 4 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/charts/keycloakx/README.md b/charts/keycloakx/README.md
index dfd1fab0..69e20bb6 100644
--- a/charts/keycloakx/README.md
+++ b/charts/keycloakx/README.md
@@ -272,11 +272,12 @@ extraEnv: |
       -XX:+UseContainerSupport
       -XX:MaxRAMPercentage=50.0
       -Djava.net.preferIPv4Stack=true
-      -Djava.awt.headless=true
 ```
 
 Alternatively one can append custom JVM options by setting the `JAVA_OPTS_APPEND` environment variable.
 
+The parameter `-Djava.awt.headless=true` is no longer required for Quarkus based Keycloak as it is set by [default](https://quarkus.io/guides/building-native-image).
+
 #### Using an External Database
 
 The Keycloak Docker image supports various database types.
diff --git a/charts/keycloakx/ci/h2-values.yaml b/charts/keycloakx/ci/h2-values.yaml
index 65402520..f56de83f 100644
--- a/charts/keycloakx/ci/h2-values.yaml
+++ b/charts/keycloakx/ci/h2-values.yaml
@@ -20,7 +20,6 @@ extraEnv: |
     value: >-
       -XX:+UseContainerSupport
       -XX:MaxRAMPercentage=50.0
-      -Djava.awt.headless=true
       -Djgroups.dns.query={{ include "keycloak.fullname" . }}-headless
 
 database:
diff --git a/charts/keycloakx/examples/postgresql-kubeping/keycloak-server-values.yaml b/charts/keycloakx/examples/postgresql-kubeping/keycloak-server-values.yaml
index 72eb2f83..fe4b9593 100644
--- a/charts/keycloakx/examples/postgresql-kubeping/keycloak-server-values.yaml
+++ b/charts/keycloakx/examples/postgresql-kubeping/keycloak-server-values.yaml
@@ -39,7 +39,6 @@ extraEnv: |
     value: >-
       -XX:+UseContainerSupport
       -XX:MaxRAMPercentage=50.0
-      -Djava.awt.headless=true
       -Dkubeping_namespace={{ .Release.Namespace }}
       -Dkubeping_label="keycloak-cluster=default"
 
diff --git a/charts/keycloakx/examples/postgresql/keycloak-server-values.yaml b/charts/keycloakx/examples/postgresql/keycloak-server-values.yaml
index 3d82fb33..159ed2f5 100644
--- a/charts/keycloakx/examples/postgresql/keycloak-server-values.yaml
+++ b/charts/keycloakx/examples/postgresql/keycloak-server-values.yaml
@@ -27,7 +27,6 @@ extraEnv: |
     value: >-
       -XX:+UseContainerSupport
       -XX:MaxRAMPercentage=50.0
-      -Djava.awt.headless=true
       -Djgroups.dns.query={{ include "keycloak.fullname" . }}-headless
 
 dbchecker: