add config value for cryostat log level

cryostatio · Oct 22, 2024 · 23c1edb · 23c1edb
1 parent d4cf55d
commit 23c1edb
Show file tree

Hide file tree

Showing 5 changed files with 36 additions and 0 deletions.
diff --git a/charts/cryostat/README.md b/charts/cryostat/README.md
@@ -57,6 +57,7 @@ helm install cryostat ./charts/cryostat
 | `core.image.tag`                                       | Tag for the main Cryostat container image                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                      | `4.0.0-snapshot`            |
 | `core.service.type`                                    | Type of Service to create for the Cryostat application                                                                                                                                                                                                                                                                                                                                                                                                                                                                                         | `ClusterIP`                 |
 | `core.service.httpPort`                                | Port number to expose on the Service for Cryostat's HTTP server                                                                                                                                                                                                                                                                                                                                                                                                                                                                                | `8181`                      |
+| `core.debug.log.level`                                 | Log level for troubleshooting and debugging                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                    | `INFO`                      |
 | `core.sslProxied`                                      | Enables SSL Proxied Environment Variables, useful when you are offloading SSL/TLS at External Loadbalancer instead of Ingress                                                                                                                                                                                                                                                                                                                                                                                                                  | `false`                     |
 | `core.ingress.enabled`                                 | Whether to create an Ingress object for the Cryostat service                                                                                                                                                                                                                                                                                                                                                                                                                                                                                   | `false`                     |
 | `core.ingress.className`                               | Ingress class name for the Cryostat application Ingress                                                                                                                                                                                                                                                                                                                                                                                                                                                                                        | `""`                        |

diff --git a/charts/cryostat/templates/cryostat_deployment.yaml b/charts/cryostat/templates/cryostat_deployment.yaml
@@ -46,6 +46,8 @@ spec:
           env:
           - name: QUARKUS_HTTP_HOST
             value: localhost
+          - name: QUARKUS_LOG_LEVEL
+            value: {{ .Values.core.debug.log.level }}
           - name: QUARKUS_HTTP_PORT
             value: "8181"
           - name: QUARKUS_HTTP_PROXY_PROXY_ADDRESS_FORWARDING

diff --git a/charts/cryostat/tests/cryostat_deployment_test.yaml b/charts/cryostat/tests/cryostat_deployment_test.yaml
@@ -61,6 +61,9 @@ tests:
       - equal:
           path: spec.template.spec.containers[?(@.name=='cryostat')].env[?(@.name=='QUARKUS_HTTP_PORT')].value
           value: "8181"
+      - equal:
+          path: spec.template.spec.containers[?(@.name=='cryostat')].env[?(@.name=='QUARKUS_LOG_LEVEL')].value
+          value: "INFO"
       - equal:
           path: spec.template.spec.containers[?(@.name=='cryostat')].env[?(@.name=='QUARKUS_HTTP_HOST')].value
           value: "localhost"
@@ -175,6 +178,17 @@ tests:
               cpu: 500m
               memory: 384Mi
 
+  - it: should set log level
+    set:
+      core:
+        debug:
+          log:
+            level: TRACE
+    asserts:
+      - equal:
+          path: spec.template.spec.containers[?(@.name=='cryostat')].env[?(@.name=='QUARKUS_LOG_LEVEL')].value
+          value: TRACE
+
   - it: should set environment variable if sidecar report generator is enabled
     set:
       reports:

diff --git a/charts/cryostat/values.schema.json b/charts/cryostat/values.schema.json
@@ -40,6 +40,21 @@
                         }
                     }
                 },
+                "debug": {
+                    "type": "object",
+                    "properties": {
+                        "log": {
+                            "type": "object",
+                            "properties": {
+                                "level": {
+                                    "type": "string",
+                                    "description": "Log level for troubleshooting and debugging",
+                                    "default": "INFO"
+                                }
+                            }
+                        }
+                    }
+                },
                 "sslProxied": {
                     "type": "boolean",
                     "description": "Enables SSL Proxied Environment Variables, useful when you are offloading SSL/TLS at External Loadbalancer instead of Ingress",

diff --git a/charts/cryostat/values.yaml b/charts/cryostat/values.yaml
@@ -13,6 +13,10 @@ core:
     type: ClusterIP
     ## @param core.service.httpPort Port number to expose on the Service for Cryostat's HTTP server
     httpPort: 8181
+  debug:
+    log:
+      ## @param core.debug.log.level Log level for troubleshooting and debugging
+      level: INFO
   ## @param core.sslProxied Enables SSL Proxied Environment Variables, useful when you are offloading SSL/TLS at External Loadbalancer instead of Ingress
   sslProxied: false
   ingress: