Repository: infonl/dimpact-zaakafhandelcomponent
Pull request #3275: chore: improve Helm Chart readme
Author: login: edgarvonk
Created: 2025-05-06T19:23:25Z | Merged: 2025-05-07T11:12:32Z

Improved Helm Chart readme.

Solves PZ-6655


Diff:
diff --git a/charts/zac/Chart.yaml b/charts/zac/Chart.yaml
index f7e00729bf..6bef89f4c6 100644
--- a/charts/zac/Chart.yaml
+++ b/charts/zac/Chart.yaml
@@ -6,7 +6,7 @@
 apiVersion: v2
 name: zaakafhandelcomponent
 description: A Helm chart for installing Zaakafhandelcomponent
-version: 1.0.58
+version: 1.0.59
 appVersion: '3.0'
 icon: https://raw.githubusercontent.com/infonl/dimpact-zaakafhandelcomponent/49f8dee60948282b546ebdfdc5cff6f8bbef0305/docs/manuals/ZAC-gebruikershandleiding/images/pic.svg
 dependencies:
diff --git a/charts/zac/README.md b/charts/zac/README.md
index 3a1613ef04..77289c602b 100644
--- a/charts/zac/README.md
+++ b/charts/zac/README.md
@@ -1,6 +1,6 @@
 # zaakafhandelcomponent
 
-![Version: 1.0.58](https://img.shields.io/badge/Version-1.0.58-informational?style=flat-square) ![AppVersion: 3.0](https://img.shields.io/badge/AppVersion-3.0-informational?style=flat-square)
+![Version: 1.0.59](https://img.shields.io/badge/Version-1.0.59-informational?style=flat-square) ![AppVersion: 3.0](https://img.shields.io/badge/AppVersion-3.0-informational?style=flat-square)
 
 A Helm chart for installing Zaakafhandelcomponent
 
@@ -48,8 +48,9 @@ The Github workflow will perform helm-linting and will bump the version if neede
 
 | Key | Type | Default | Description |
 |-----|------|---------|-------------|
+| additionalAllowedFileTypes | string | `nil` | An optional list of additional file extensions that can be uploaded |
 | affinity | object | `{}` | set affinity parameters |
-| auth.clientId | string | `""` | Client ID and secret as defined in the realm |
+| auth.clientId | string | `""` | Client ID and secret as defined in the Keycloak ZAC realm |
 | auth.realm | string | `""` |  |
 | auth.secret | string | `""` |  |
 | auth.server | string | `""` |  |
@@ -65,14 +66,14 @@ The Github workflow will perform helm-linting and will bump the version if neede
 | brpApi.protocollering.originOin | string | `""` | If specified, enables the BRP Protocollering |
 | brpApi.protocollering.verwerking | string | `"zaakafhandelcomponent"` | Verwerking for BRP Protocollering |
 | brpApi.url | string | `""` |  |
-| catalogusDomein | string | `"ALG"` | OpenZaak Catalogus Domein |
+| catalogusDomein | string | `"ALG"` | ZAC OpenZaak Catalogus Domein |
 | contextUrl | string | `""` | External URL to the zaakafhandelcomponent. (https://zaakafhandelcomponent.example.com) |
-| db.host | string | `""` |  |
+| db.host | string | `""` | database.internal or 1.2.3.4 |
 | db.name | string | `""` |  |
 | db.password | string | `""` |  |
 | db.user | string | `""` |  |
 | extraDeploy | list | `[]` | Extra objects to deploy (value evaluated as a template) |
-| featureFlags.bpmnSupport | bool | `false` |  |
+| featureFlags.bpmnSupport | bool | `false` | turns BPMN support on or off |
 | fullnameOverride | string | `""` | fullname to use |
 | gemeente.code | string | `""` |  |
 | gemeente.mail | string | `""` |  |
@@ -88,17 +89,18 @@ The Github workflow will perform helm-linting and will bump the version if neede
 | ingress.hosts[0].paths[0].path | string | `"/"` |  |
 | ingress.hosts[0].paths[0].pathType | string | `"ImplementationSpecific"` |  |
 | ingress.tls | list | `[]` |  |
-| initContainer | object | `{"enabled":true,"image":{"repository":"curlimages/curl","tag":"8.13.0@sha256:d43bdb28bae0be0998f3be83199bfb2b81e0a30b034b6d7586ce7e05de34c3fd"}}` | set initContainer parameters |
+| initContainer.enabled | bool | `true` |  |
+| initContainer.image.repository | string | `"curlimages/curl"` |  |
+| initContainer.image.tag | string | `"8.13.0@sha256:d43bdb28bae0be0998f3be83199bfb2b81e0a30b034b6d7586ce7e05de34c3fd"` |  |
 | keycloak.adminClient.id | string | `""` | Keycloak ZAC admin client name |
 | keycloak.adminClient.secret | string | `""` | Keycloak ZAC admin client secret |
 | klantinteractiesApi.token | string | `""` |  |
 | klantinteractiesApi.url | string | `""` |  |
 | kvkApi.apiKey | string | `""` |  |
 | kvkApi.url | string | `""` |  |
-| mail | object | `{"smtp":{"password":"","port":"587","server":"","username":""}}` | Email sending connection. SPF record needs to be properly setup in DNS |
 | mail.smtp.password | string | `""` | SMTP server password if authentication is required. Optional |
 | mail.smtp.port | string | `"587"` | SMTP server port: 587 for TLS, port 25 for relaying. Required |
-| mail.smtp.server | string | `""` | SMTP server host (for example localhost or in-v3.mailjet.com). Required |
+| mail.smtp.server | string | `""` | SMTP server host (for example, localhost or in-v3.mailjet.com). Required |
 | mail.smtp.username | string | `""` | SMTP server username if authentication is required. Optional |
 | maxFileSizeMB | int | `80` | Maximum size (in Mega Bytes) of files that can be uploaded. |
 | nameOverride | string | `""` | name to use |
@@ -185,7 +187,7 @@ The Github workflow will perform helm-linting and will bump the version if neede
 | nginx.service.type | string | `"ClusterIP"` |  |
 | nginx.useXForwardedHost | bool | `false` |  |
 | nodeSelector | object | `{}` | set node selector parameters |
-| notificationsSecretKey | string | `""` |  |
+| notificationsSecretKey | string | `""` | API key for the ZGW Notificaties Consumer API integration; also needs to be configured in Open Notificaties |
 | objectenApi.token | string | `""` |  |
 | objectenApi.url | string | `""` |  |
 | office_converter.affinity | object | `{}` |  |
@@ -213,7 +215,7 @@ The Github workflow will perform helm-linting and will bump the version if neede
 | opa.image.repository | string | `"openpolicyagent/opa"` |  |
 | opa.image.tag | string | `"1.4.2-static@sha256:3c995dc8a59f6ddfd92eb7404d2f7ff9fe71cd025d9251199957a8a6afbfd76e"` |  |
 | opa.imagePullSecrets | list | `[]` |  |
-| opa.name | string | `"opa"` | set url if the opa url cannot be automatically determined and is not run as a sidecar. the opa url should be the url the openpolicyagent can be reached on from ZAC ( for example: http://release-opa.default.svc.cluster.local:8181 ) url: "" |
+| opa.name | string | `"opa"` |  |
 | opa.nodeSelector | object | `{}` |  |
 | opa.podAnnotations | object | `{}` |  |
 | opa.podSecurityContext | object | `{}` |  |
@@ -226,7 +228,7 @@ The Github workflow will perform helm-linting and will bump the version if neede
 | opa.service.type | string | `"ClusterIP"` |  |
 | opa.sidecar | bool | `false` | set sidecar to true to run the opa service together with the zac pod |
 | opa.tolerations | list | `[]` |  |
-| openForms.url | string | `""` |  |
+| openForms.url | string | `""` | Not used at the moment. |
 | opentelemetry-collector.config.receivers.jaeger | object | `{}` |  |
 | opentelemetry-collector.config.receivers.prometheus | object | `{}` |  |
 | opentelemetry-collector.config.receivers.zipkin | object | `{}` |  |
@@ -244,15 +246,17 @@ The Github workflow will perform helm-linting and will bump the version if neede
 | opentelemetry-collector.ports.zipkin.enabled | bool | `false` |  |
 | opentelemetry-collector.presets.clusterMetrics.enabled | bool | `false` |  |
 | opentelemetry-collector.replicaCount | int | `1` |  |
+| opentelemetry_zaakafhandelcomponent.disabled | string | `"-true"` | Enables or disables the ZAC OpenTelemetry integration. Disabled by default. |
+| opentelemetry_zaakafhandelcomponent.endpoint | string | `""` | OpenTelemetry collector endpoint URL |
 | organizations.bron.rsin | string | `""` | The RSIN of the Non-natural person - the organization that created the zaak. Must be a valid RSIN of 9 numbers and comply with https://nl.wikipedia.org/wiki/Burgerservicenummer#11-proef |
 | organizations.verantwoordelijke.rsin | string | `""` | The RSIN of the Non-natural person - the organization that is ultimately responsible for handling a zaak or establishing a decision. Must be a valid RSIN of 9 numbers and comply with https://nl.wikipedia.org/wiki/Burgerservicenummer#11-proef |
 | podAnnotations | object | `{}` | pod specific annotations |
 | podSecurityContext | object | `{}` | pod specific security context |
 | remoteDebug | bool | `false` | Enable Java remote debugging |
-| replicaCount | int | `1` | the number of replicas to run |
+| replicaCount | int | `1` | The number of replicas to run |
 | resources.requests.cpu | string | `"100m"` |  |
 | resources.requests.memory | string | `"1Gi"` |  |
-| securityContext | object | `{}` |  |
+| securityContext | object | `{}` | generic security context |
 | service.annotations | object | `{}` |  |
 | service.port | int | `80` |  |
 | service.type | string | `"ClusterIP"` |  |
@@ -278,7 +282,7 @@ The Github workflow will perform helm-linting and will bump the version if neede
 | signaleringen.tolerations | list | `[]` |  |
 | smartDocuments.authentication | string | `""` | Authentication token |
 | smartDocuments.enabled | bool | `false` | Enable SmartDocuments integration for creating a new document |
-| smartDocuments.fixedUserName | string | `""` | If this setting is set, then templates in SmartDocuments cannot use user-specific values. |
+| smartDocuments.fixedUserName | string | `""` | If set this overrides the sending of the username of the user that is logged in to ZAC to SmartDocuments with a fixed value. This username is sent to SmartDocuments when creating a new document as an HTTP header. For most target environments this should not be set, assuming that all users that are available in ZAC are also available in the SmartDocuments environment with the same username. If this setting is set, then templates in SmartDocuments cannot use user-specific values. |
 | smartDocuments.url | string | `""` | URL to SmartDocuments instance. For example: https://partners.smartdocuments.com |
 | solr-operator.affinity | object | `{}` | affinity for solr-operator |
 | solr-operator.annotations | object | `{}` | annotations for solr-operator |
diff --git a/charts/zac/values.yaml b/charts/zac/values.yaml
index 1f530545ee..a71107f125 100644
--- a/charts/zac/values.yaml
+++ b/charts/zac/values.yaml
@@ -7,25 +7,30 @@
 # This is a YAML-formatted file.
 # Declare variables to be passed into your templates.
 
+# Please add Helm Docs field descriptions using the format `# -- blah` to leaf nodes only and not to parent nodes, except if all leaf nodes
+# have field descriptions. Otherwise, leaf nodes with field descriptions will be ignored and will not be included in the generated README.md.
+# See: https://github.com/norwoodj/helm-docs for details.
+
 # -- External URL to the zaakafhandelcomponent. (https://zaakafhandelcomponent.example.com)
 contextUrl: ""
 
-# db Configuration of database connection
+# Configuration of the ZAC database connection
 db:
-  host: ""  # -- database.internal or 1.2.3.4
+  # -- database.internal or 1.2.3.4
+  host: ""
   name: ""
   user: ""
   password: ""
 
-# auth Configuration of OpenID Connect
+# Configuration for the Keycloak OpenID Connect integration
 auth:
-  server: ""  # https://keycloak.example.com
+  server: ""
   realm: ""
-  # -- Client ID and secret as defined in the realm
+  # -- Client ID and secret as defined in the Keycloak ZAC realm
   clientId: ""
   secret: ""
 
-# configuration of the ZAC admin client in Keycloak
+# Configuration of the ZAC admin client in Keycloak
 keycloak:
   adminClient:
     # -- Keycloak ZAC admin client name
@@ -33,16 +38,16 @@ keycloak:
     # -- Keycloak ZAC admin client secret
     secret: ""
 
-# gemeente Configuration gemeente
+# Council specific configuration
 gemeente:
   code: ""
   naam: ""
   mail: ""
 
-# -- Email sending connection. SPF record needs to be properly setup in DNS
+# SMTP provider configuration. SPF record needs to be properly set up in DNS
 mail:
   smtp:
-    # -- SMTP server host (for example localhost or in-v3.mailjet.com). Required
+    # -- SMTP server host (for example, localhost or in-v3.mailjet.com). Required
     server: ""
     # -- SMTP server port: 587 for TLS, port 25 for relaying. Required
     port: "587"
@@ -51,33 +56,34 @@ mail:
     # -- SMTP server password if authentication is required. Optional
     password: ""
 
-# zgwApis Configuration of ZGW API's provider (Open Zaak)
+# ZGW API configuration for integration with the ZGW APIs provider (OpenZaak)
 zgwApis:
-  url: ""  # http://open-zaak.internal
-  urlExtern: ""  # https://open-zaak.example.com
+  url: ""
+  urlExtern: ""
   clientId: ""
   secret: ""
 
-# API key for the ZGW Notificaties Consumer API - ZAC integration; also needs to be configured in Open Notificaties
+# -- API key for the ZGW Notificaties Consumer API integration; also needs to be configured in Open Notificaties
 notificationsSecretKey: ""
 
-# klantenApi Configuration of Klanten API provider (Open Klant)
+# Integration with the Klantinteracties API provider (OpenKlant)
 klantinteractiesApi:
   url: ""  # http://open-klant.internal/klantinteracties
   token: ""
 
-# featureFlags defines supported feature flags and their default values
+# Supported feature flags
 featureFlags:
+  # -- turns BPMN support on or off
   bpmnSupport: false
 
-# objectenApi Configuration of the Objecten API provider (Overige Registraties)
+# Integration with the Objecten API provider
 objectenApi:
-  url: ""  # http://objecten.internal
+  url: ""
   token: ""
 
-# brpApi Configuration of the BRP Api provider (Haal Centraal)
+# Integration with the BRP API provider (Haal Centraal)
 brpApi:
-  url: ""  # https://www.haalcentraal.nl/haalcentraal
+  url: ""
   apiKey: ""
   protocollering:
     # -- If specified, enables the BRP Protocollering
@@ -87,19 +93,20 @@ brpApi:
     # -- Verwerking for BRP Protocollering
     verwerking: "zaakafhandelcomponent"
 
-# Configuration of the KVK API provider (KVK)
+# Integration with the KVK API provider (KVK)
 kvkApi:
   url: ""  # https://developers.kvk.nl/test
   apiKey: ""
 
-# Configuration of the BAG API provider (Kadaster)
+# Integration with the BAG API provider (Kadaster)
 bagApi:
   url: ""  # https://api.bag.kadaster.nl/lvbag/individuelebevragingen/v2
   apiKey: ""
 
-# Open Formulieren configuration of the Open Forms URL
+# Integration with Open Formulieren.
 openForms:
-  url: ""  # https://open-forms.example.com
+  # -- Not used at the moment.
+  url: ""
 
 # -- API key for authentication of internal ZAC endpoints
 zacInternalEndpointsApiKey: ""
@@ -111,15 +118,14 @@ smartDocuments:
   url: ""
   # -- Authentication token
   authentication: ""
-  # -- If set this overrides the sending of the username of the user that is logged in to ZAC to
-  # -- SmartDocuments with a fixed value.
-  # -- This username is sent to SmartDocuments when creating a new document as an HTTP header.
-  # -- For most target environments this should not be set, assuming that
-  # -- all users that are available in ZAC are also available in the SmartDocuments environment with the same username.
-  # -- If this setting is set, then templates in SmartDocuments cannot use user-specific values.
+  # -- If set this overrides the sending of the username of the user that is logged in to ZAC to SmartDocuments with a fixed value.
+  # This username is sent to SmartDocuments when creating a new document as an HTTP header.
+  # For most target environments this should not be set, assuming that all users that are available in ZAC are also available in the
+  # SmartDocuments environment with the same username.
+  # If this setting is set, then templates in SmartDocuments cannot use user-specific values.
   fixedUserName: ""
 
-# signaleringen Configuration of the signaleren job
+# Signaleringen cronjob configuration
 signaleringen:
   # -- Delete any signaleringen older than this number of days when the corresponding admin endpoint is called.
   deleteOlderThanDays: "14"
@@ -144,25 +150,27 @@ signaleringen:
   securityContext: {}
   tolerations: []
 
-# ZAC OpenTelemetry configuration. Only read when opentelemetry-collector is enabled
-# opentelemetry_zaakafhandelcomponent:
-#   disabled: "-true"
-#   endpoint: "http://otel-collector:4317"
+# OpenTelemetry configuration. Only read when opentelemetry-collector is enabled
+opentelemetry_zaakafhandelcomponent:
+  # -- Enables or disables the ZAC OpenTelemetry integration. Disabled by default.
+  disabled: "-true"
+  # -- OpenTelemetry collector endpoint URL
+  endpoint: ""
 
 
 # -- Maximum size (in Mega Bytes) of files that can be uploaded.
 maxFileSizeMB: 80
 
 # -- An optional list of additional file extensions that can be uploaded
-# additionalAllowedFileTypes: aaa,bbb,ccc
+additionalAllowedFileTypes:
 
 # -- Enable Java remote debugging
 remoteDebug: false
 
-# -- jvm options to set in the _JAVA_OPTIONS env var, recommended defaults are used if not provided and are shown below
+# -- JVM options to set in the _JAVA_OPTIONS env var, recommended defaults are used if not provided and are shown below
 # javaOptions: -Xmx1024m -Xms1024m -Xlog:gc::time,uptime
 
-# -- the number of replicas to run
+# -- The number of replicas to run
 replicaCount: 1
 
 # image the image parameters to use
@@ -197,7 +205,7 @@ podAnnotations: {}
 podSecurityContext: {}
 # fsGroup: 2000
 
-# generic security context
+# -- generic security context
 securityContext: {}
   # capabilities:
   #   drop:
@@ -233,10 +241,10 @@ ingress:
 backendConfig:
   enabled: false
 
-# specify resources limits and requests if needed, prefilled values are the minimum recommended
+# specify resource limits and requests if needed, prefilled values are the minimum recommended
 resources:
-  # We usually recommend not to specify default resources and to leave this as a conscious
-  # choice for the user. This also increases chances charts run on environments with little
+  # We usually recommend not specifying default resources and to leave this as a conscious
+  # choice for the user. This also increases the chances charts run on environments with little
   # resources, such as Minikube. If you do want to specify resources, uncomment the following
   # lines, adjust them as necessary, and remove the curly braces after 'resources:'.
   # limits:
@@ -246,7 +254,7 @@ resources:
     cpu: 100m
     memory: 1Gi
 
-# set autoscaling parameters
+# autoscaling parameters
 autoscaling:
   enabled: false
   minReplicas: 1
@@ -263,19 +271,19 @@ tolerations: []
 # -- set affinity parameters
 affinity: {}
 
-# -- set initContainer parameters
+# set initContainer parameters
 initContainer:
   enabled: true
   image:
     repository: curlimages/curl
     tag: 8.13.0@sha256:d43bdb28bae0be0998f3be83199bfb2b81e0a30b034b6d7586ce7e05de34c3fd
 
-# opa provision the opa deployment, prefilled resources values are the minimum recommended
+# Open Policy Agent (OPA) configuration, prefilled resources values are the minimum recommended
 opa:
   enabled: true
   # -- set sidecar to true to run the opa service together with the zac pod
   sidecar: false
-  # -- set url if the opa url cannot be automatically determined and is not run as a sidecar. the opa url should be the url the openpolicyagent can be reached on from ZAC ( for example: http://release-opa.default.svc.cluster.local:8181 )
+  # set url if the opa url cannot be automatically determined and is not run as a sidecar. the opa url should be the url the openpolicyagent can be reached on from ZAC ( for example: http://release-opa.default.svc.cluster.local:8181 )
   # url: ""
   name: "opa"
   service:
@@ -315,7 +323,7 @@ opa:
 # See the zaakafhandelcomponent wiki on more information about this.
 #
 
-# solr enable to use the included solr helm chart to provision a solr to use with zac
+# Solr enable to use the included solr helm chart to provision a solr to use with zac
 solr:
   # -- set enabled to true to provision bitnami solr version with the zac core
   enabled: false
@@ -511,7 +519,7 @@ solr-operator:
       # -- tolerations for zookeeper
       tolerations: []
 
-# office_converter use the office converter specified, prefilled resources values are the minimum recommended
+# office_converter configuration. Prefilled resources values are the minimum recommended
 office_converter:
   enabled: true
   # set url if the office-converter url cannot be automatically determined
@@ -692,5 +700,5 @@ organizations:
     # -- The RSIN of the Non-natural person - the organization that created the zaak. Must be a valid RSIN of 9 numbers and comply with https://nl.wikipedia.org/wiki/Burgerservicenummer#11-proef
     rsin: ""
 
-# -- OpenZaak Catalogus Domein
+# -- ZAC OpenZaak Catalogus Domein
 catalogusDomein: ALG


Reviews:
user: login: copilot-pull-request-reviewer[bot]
body: ## Pull Request Overview

This PR improves the Helm Chart readme by enhancing configuration comments and clarifying parameter descriptions in the charts/zac/values.yaml file.  
- Updated and standardized configuration comments to improve readability  
- Enhanced guidance within the YAML file, including minor updates to image versions and phrasing




state: COMMENTED
author_association: CONTRIBUTOR
submitted_at: 2025-05-06T19:23:34Z
user: login: niorg
state: COMMENTED
author_association: MEMBER
submitted_at: 2025-05-06T19:54:44Z
user: login: edgarvonk
state: COMMENTED
author_association: MEMBER
submitted_at: 2025-05-07T07:19:24Z
user: login: edgarvonk
state: COMMENTED
author_association: MEMBER
submitted_at: 2025-05-07T07:21:15Z
user: login: niorg
state: COMMENTED
author_association: MEMBER
submitted_at: 2025-05-07T07:23:58Z
user: login: edgarvonk
state: COMMENTED
author_association: MEMBER
submitted_at: 2025-05-07T07:40:58Z
user: login: edgarvonk
state: COMMENTED
author_association: MEMBER
submitted_at: 2025-05-07T07:42:49Z
user: login: niorg
state: COMMENTED
author_association: MEMBER
submitted_at: 2025-05-07T08:00:27Z
user: login: edgarvonk
state: COMMENTED
author_association: MEMBER
submitted_at: 2025-05-07T08:14:00Z
user: login: edgarvonk
state: COMMENTED
author_association: MEMBER
submitted_at: 2025-05-07T08:15:51Z
user: login: niorg
state: COMMENTED
author_association: MEMBER
submitted_at: 2025-05-07T08:24:40Z
user: login: edgarvonk
state: COMMENTED
author_association: MEMBER
submitted_at: 2025-05-07T08:49:17Z
user: login: edgarvonk
state: COMMENTED
author_association: MEMBER
submitted_at: 2025-05-07T08:52:37Z
user: login: niorg
state: COMMENTED
author_association: MEMBER
submitted_at: 2025-05-07T08:54:05Z
user: login: edgarvonk
state: COMMENTED
author_association: MEMBER
submitted_at: 2025-05-07T09:15:29Z
user: login: edgarvonk
state: COMMENTED
author_association: MEMBER
submitted_at: 2025-05-07T09:48:02Z
user: login: edgarvonk
state: COMMENTED
author_association: MEMBER
submitted_at: 2025-05-07T10:09:48Z
user: login: niorg
state: COMMENTED
author_association: MEMBER
submitted_at: 2025-05-07T10:13:49Z
user: login: edgarvonk
state: COMMENTED
author_association: MEMBER
submitted_at: 2025-05-07T10:18:00Z
user: login: edgarvonk
state: COMMENTED
author_association: MEMBER
submitted_at: 2025-05-07T10:20:53Z
user: login: niorg
state: COMMENTED
author_association: MEMBER
submitted_at: 2025-05-07T10:39:31Z
user: login: niorg
state: APPROVED
author_association: MEMBER
submitted_at: 2025-05-07T10:44:54Z
user: login: edgarvonk
state: COMMENTED
author_association: MEMBER
submitted_at: 2025-05-07T11:03:32Z

Review comments:
diff_hunk: @@ -48,57 +48,42 @@ The Github workflow will perform helm-linting and will bump the version if neede
 
 | Key | Type | Default | Description |
 |-----|------|---------|-------------|
+| additionalAllowedFileTypes | string | `nil` | An optional list of additional file extensions that can be uploaded |
 | affinity | object | `{}` | set affinity parameters |
-| auth.clientId | string | `""` | Client ID and secret as defined in the realm |
-| auth.realm | string | `""` |  |
-| auth.secret | string | `""` |  |
-| auth.server | string | `""` |  |
-| autoscaling.enabled | bool | `false` |  |
-| autoscaling.maxReplicas | int | `100` |  |
-| autoscaling.minReplicas | int | `1` |  |
-| autoscaling.targetCPUUtilizationPercentage | int | `80` |  |
-| backendConfig.enabled | bool | `false` |  |
-| bagApi.apiKey | string | `""` |  |
-| bagApi.url | string | `""` |  |
+| auth | object | `{"clientId":"","realm":"","secret":"","server":""}` | Configuration for the Keycloak OpenID Connect integration |
+| auth.clientId | string | `""` | Client ID and secret as defined in the Keycloak ZAC realm |
+| autoscaling | object | `{"enabled":false,"maxReplicas":100,"minReplicas":1,"targetCPUUtilizationPercentage":80}` | autoscaling parameters |
path: charts/zac/README.md
user: login: niorg
body: Looks like only the object is now described, not the keys inside. Don't really like that.
created_at: 2025-05-06T19:54:44Z
author_association: MEMBER
diff_hunk: @@ -48,57 +48,42 @@ The Github workflow will perform helm-linting and will bump the version if neede
 
 | Key | Type | Default | Description |
 |-----|------|---------|-------------|
+| additionalAllowedFileTypes | string | `nil` | An optional list of additional file extensions that can be uploaded |
 | affinity | object | `{}` | set affinity parameters |
-| auth.clientId | string | `""` | Client ID and secret as defined in the realm |
-| auth.realm | string | `""` |  |
-| auth.secret | string | `""` |  |
-| auth.server | string | `""` |  |
-| autoscaling.enabled | bool | `false` |  |
-| autoscaling.maxReplicas | int | `100` |  |
-| autoscaling.minReplicas | int | `1` |  |
-| autoscaling.targetCPUUtilizationPercentage | int | `80` |  |
-| backendConfig.enabled | bool | `false` |  |
-| bagApi.apiKey | string | `""` |  |
-| bagApi.url | string | `""` |  |
+| auth | object | `{"clientId":"","realm":"","secret":"","server":""}` | Configuration for the Keycloak OpenID Connect integration |
+| auth.clientId | string | `""` | Client ID and secret as defined in the Keycloak ZAC realm |
+| autoscaling | object | `{"enabled":false,"maxReplicas":100,"minReplicas":1,"targetCPUUtilizationPercentage":80}` | autoscaling parameters |
path: charts/zac/README.md
user: login: edgarvonk
body: Is that so? In the example above I do see a description for e.g. `auth.clientId` (as well as for `auth` itself)?


created_at: 2025-05-07T07:19:24Z
author_association: MEMBER
diff_hunk: @@ -48,57 +48,42 @@ The Github workflow will perform helm-linting and will bump the version if neede
 
 | Key | Type | Default | Description |
 |-----|------|---------|-------------|
+| additionalAllowedFileTypes | string | `nil` | An optional list of additional file extensions that can be uploaded |
 | affinity | object | `{}` | set affinity parameters |
-| auth.clientId | string | `""` | Client ID and secret as defined in the realm |
-| auth.realm | string | `""` |  |
-| auth.secret | string | `""` |  |
-| auth.server | string | `""` |  |
-| autoscaling.enabled | bool | `false` |  |
-| autoscaling.maxReplicas | int | `100` |  |
-| autoscaling.minReplicas | int | `1` |  |
-| autoscaling.targetCPUUtilizationPercentage | int | `80` |  |
-| backendConfig.enabled | bool | `false` |  |
-| bagApi.apiKey | string | `""` |  |
-| bagApi.url | string | `""` |  |
+| auth | object | `{"clientId":"","realm":"","secret":"","server":""}` | Configuration for the Keycloak OpenID Connect integration |
+| auth.clientId | string | `""` | Client ID and secret as defined in the Keycloak ZAC realm |
+| autoscaling | object | `{"enabled":false,"maxReplicas":100,"minReplicas":1,"targetCPUUtilizationPercentage":80}` | autoscaling parameters |
path: charts/zac/README.md
user: login: edgarvonk
body: I see lots of descriptions of keys in the README.md. Not all because not all have a field description but that was already the case. But thanks to this PR we have a lot more descriptions now.
created_at: 2025-05-07T07:21:15Z
author_association: MEMBER
diff_hunk: @@ -48,57 +48,42 @@ The Github workflow will perform helm-linting and will bump the version if neede
 
 | Key | Type | Default | Description |
 |-----|------|---------|-------------|
+| additionalAllowedFileTypes | string | `nil` | An optional list of additional file extensions that can be uploaded |
 | affinity | object | `{}` | set affinity parameters |
-| auth.clientId | string | `""` | Client ID and secret as defined in the realm |
-| auth.realm | string | `""` |  |
-| auth.secret | string | `""` |  |
-| auth.server | string | `""` |  |
-| autoscaling.enabled | bool | `false` |  |
-| autoscaling.maxReplicas | int | `100` |  |
-| autoscaling.minReplicas | int | `1` |  |
-| autoscaling.targetCPUUtilizationPercentage | int | `80` |  |
-| backendConfig.enabled | bool | `false` |  |
-| bagApi.apiKey | string | `""` |  |
-| bagApi.url | string | `""` |  |
+| auth | object | `{"clientId":"","realm":"","secret":"","server":""}` | Configuration for the Keycloak OpenID Connect integration |
+| auth.clientId | string | `""` | Client ID and secret as defined in the Keycloak ZAC realm |
+| autoscaling | object | `{"enabled":false,"maxReplicas":100,"minReplicas":1,"targetCPUUtilizationPercentage":80}` | autoscaling parameters |
path: charts/zac/README.md
user: login: niorg
body: Check `autoscaling` or `office_converter` which are way less descriptive now
created_at: 2025-05-07T07:23:58Z
author_association: MEMBER
diff_hunk: @@ -48,57 +48,42 @@ The Github workflow will perform helm-linting and will bump the version if neede
 
 | Key | Type | Default | Description |
 |-----|------|---------|-------------|
+| additionalAllowedFileTypes | string | `nil` | An optional list of additional file extensions that can be uploaded |
 | affinity | object | `{}` | set affinity parameters |
-| auth.clientId | string | `""` | Client ID and secret as defined in the realm |
-| auth.realm | string | `""` |  |
-| auth.secret | string | `""` |  |
-| auth.server | string | `""` |  |
-| autoscaling.enabled | bool | `false` |  |
-| autoscaling.maxReplicas | int | `100` |  |
-| autoscaling.minReplicas | int | `1` |  |
-| autoscaling.targetCPUUtilizationPercentage | int | `80` |  |
-| backendConfig.enabled | bool | `false` |  |
-| bagApi.apiKey | string | `""` |  |
-| bagApi.url | string | `""` |  |
+| auth | object | `{"clientId":"","realm":"","secret":"","server":""}` | Configuration for the Keycloak OpenID Connect integration |
+| auth.clientId | string | `""` | Client ID and secret as defined in the Keycloak ZAC realm |
+| autoscaling | object | `{"enabled":false,"maxReplicas":100,"minReplicas":1,"targetCPUUtilizationPercentage":80}` | autoscaling parameters |
path: charts/zac/README.md
user: login: edgarvonk
body: I don't understand.. I actually see more documentation for `office_converter` now. Previously there was no description for the object at all but now there is. There were no key field descriptions there and there still aren't.
created_at: 2025-05-07T07:40:57Z
author_association: MEMBER
diff_hunk: @@ -48,57 +48,42 @@ The Github workflow will perform helm-linting and will bump the version if neede
 
 | Key | Type | Default | Description |
 |-----|------|---------|-------------|
+| additionalAllowedFileTypes | string | `nil` | An optional list of additional file extensions that can be uploaded |
 | affinity | object | `{}` | set affinity parameters |
-| auth.clientId | string | `""` | Client ID and secret as defined in the realm |
-| auth.realm | string | `""` |  |
-| auth.secret | string | `""` |  |
-| auth.server | string | `""` |  |
-| autoscaling.enabled | bool | `false` |  |
-| autoscaling.maxReplicas | int | `100` |  |
-| autoscaling.minReplicas | int | `1` |  |
-| autoscaling.targetCPUUtilizationPercentage | int | `80` |  |
-| backendConfig.enabled | bool | `false` |  |
-| bagApi.apiKey | string | `""` |  |
-| bagApi.url | string | `""` |  |
+| auth | object | `{"clientId":"","realm":"","secret":"","server":""}` | Configuration for the Keycloak OpenID Connect integration |
+| auth.clientId | string | `""` | Client ID and secret as defined in the Keycloak ZAC realm |
+| autoscaling | object | `{"enabled":false,"maxReplicas":100,"minReplicas":1,"targetCPUUtilizationPercentage":80}` | autoscaling parameters |
path: charts/zac/README.md
user: login: edgarvonk
body: Same with `autoscaling` I think.
created_at: 2025-05-07T07:42:49Z
author_association: MEMBER
diff_hunk: @@ -48,57 +48,42 @@ The Github workflow will perform helm-linting and will bump the version if neede
 
 | Key | Type | Default | Description |
 |-----|------|---------|-------------|
+| additionalAllowedFileTypes | string | `nil` | An optional list of additional file extensions that can be uploaded |
 | affinity | object | `{}` | set affinity parameters |
-| auth.clientId | string | `""` | Client ID and secret as defined in the realm |
-| auth.realm | string | `""` |  |
-| auth.secret | string | `""` |  |
-| auth.server | string | `""` |  |
-| autoscaling.enabled | bool | `false` |  |
-| autoscaling.maxReplicas | int | `100` |  |
-| autoscaling.minReplicas | int | `1` |  |
-| autoscaling.targetCPUUtilizationPercentage | int | `80` |  |
-| backendConfig.enabled | bool | `false` |  |
-| bagApi.apiKey | string | `""` |  |
-| bagApi.url | string | `""` |  |
+| auth | object | `{"clientId":"","realm":"","secret":"","server":""}` | Configuration for the Keycloak OpenID Connect integration |
+| auth.clientId | string | `""` | Client ID and secret as defined in the Keycloak ZAC realm |
+| autoscaling | object | `{"enabled":false,"maxReplicas":100,"minReplicas":1,"targetCPUUtilizationPercentage":80}` | autoscaling parameters |
path: charts/zac/README.md
user: login: niorg
body: No, there were individual entries for each subkey before. Without description yes, but at least much more readable than the json object blob it is now documenting in the README
created_at: 2025-05-07T08:00:27Z
author_association: MEMBER
diff_hunk: @@ -48,57 +48,42 @@ The Github workflow will perform helm-linting and will bump the version if neede
 
 | Key | Type | Default | Description |
 |-----|------|---------|-------------|
+| additionalAllowedFileTypes | string | `nil` | An optional list of additional file extensions that can be uploaded |
 | affinity | object | `{}` | set affinity parameters |
-| auth.clientId | string | `""` | Client ID and secret as defined in the realm |
-| auth.realm | string | `""` |  |
-| auth.secret | string | `""` |  |
-| auth.server | string | `""` |  |
-| autoscaling.enabled | bool | `false` |  |
-| autoscaling.maxReplicas | int | `100` |  |
-| autoscaling.minReplicas | int | `1` |  |
-| autoscaling.targetCPUUtilizationPercentage | int | `80` |  |
-| backendConfig.enabled | bool | `false` |  |
-| bagApi.apiKey | string | `""` |  |
-| bagApi.url | string | `""` |  |
+| auth | object | `{"clientId":"","realm":"","secret":"","server":""}` | Configuration for the Keycloak OpenID Connect integration |
+| auth.clientId | string | `""` | Client ID and secret as defined in the Keycloak ZAC realm |
+| autoscaling | object | `{"enabled":false,"maxReplicas":100,"minReplicas":1,"targetCPUUtilizationPercentage":80}` | autoscaling parameters |
path: charts/zac/README.md
user: login: edgarvonk
body: Then I don't understand how this works.. :-(  In https://github.com/losisin/helm-docs-github-action and https://github.com/norwoodj/helm-docs I read that you need to use the Helm chart field descriptions using the syntax `# -- blahblah` and that's all I did. 
created_at: 2025-05-07T08:14:00Z
author_association: MEMBER
diff_hunk: @@ -48,57 +48,42 @@ The Github workflow will perform helm-linting and will bump the version if neede
 
 | Key | Type | Default | Description |
 |-----|------|---------|-------------|
+| additionalAllowedFileTypes | string | `nil` | An optional list of additional file extensions that can be uploaded |
 | affinity | object | `{}` | set affinity parameters |
-| auth.clientId | string | `""` | Client ID and secret as defined in the realm |
-| auth.realm | string | `""` |  |
-| auth.secret | string | `""` |  |
-| auth.server | string | `""` |  |
-| autoscaling.enabled | bool | `false` |  |
-| autoscaling.maxReplicas | int | `100` |  |
-| autoscaling.minReplicas | int | `1` |  |
-| autoscaling.targetCPUUtilizationPercentage | int | `80` |  |
-| backendConfig.enabled | bool | `false` |  |
-| bagApi.apiKey | string | `""` |  |
-| bagApi.url | string | `""` |  |
+| auth | object | `{"clientId":"","realm":"","secret":"","server":""}` | Configuration for the Keycloak OpenID Connect integration |
+| auth.clientId | string | `""` | Client ID and secret as defined in the Keycloak ZAC realm |
+| autoscaling | object | `{"enabled":false,"maxReplicas":100,"minReplicas":1,"targetCPUUtilizationPercentage":80}` | autoscaling parameters |
path: charts/zac/README.md
user: login: edgarvonk
body: PS: I don't see those JSON object blobs in the new readme myself but maybe I'm looking in the wrong place? See e.g. https://github.com/infonl/dimpact-zaakafhandelcomponent/pull/3275/files?short_path=8e48544#diff-8e48544b6b97d7aa113b0eee9ca7f1219ab6a0d1084dfb208a1209592e32a4ba
created_at: 2025-05-07T08:15:51Z
author_association: MEMBER
diff_hunk: @@ -48,57 +48,42 @@ The Github workflow will perform helm-linting and will bump the version if neede
 
 | Key | Type | Default | Description |
 |-----|------|---------|-------------|
+| additionalAllowedFileTypes | string | `nil` | An optional list of additional file extensions that can be uploaded |
 | affinity | object | `{}` | set affinity parameters |
-| auth.clientId | string | `""` | Client ID and secret as defined in the realm |
-| auth.realm | string | `""` |  |
-| auth.secret | string | `""` |  |
-| auth.server | string | `""` |  |
-| autoscaling.enabled | bool | `false` |  |
-| autoscaling.maxReplicas | int | `100` |  |
-| autoscaling.minReplicas | int | `1` |  |
-| autoscaling.targetCPUUtilizationPercentage | int | `80` |  |
-| backendConfig.enabled | bool | `false` |  |
-| bagApi.apiKey | string | `""` |  |
-| bagApi.url | string | `""` |  |
+| auth | object | `{"clientId":"","realm":"","secret":"","server":""}` | Configuration for the Keycloak OpenID Connect integration |
+| auth.clientId | string | `""` | Client ID and secret as defined in the Keycloak ZAC realm |
+| autoscaling | object | `{"enabled":false,"maxReplicas":100,"minReplicas":1,"targetCPUUtilizationPercentage":80}` | autoscaling parameters |
path: charts/zac/README.md
user: login: niorg
body: I think we don't need a description for the main key, only the subkeys. 

This is what I see in your change: 
https://github.com/infonl/dimpact-zaakafhandelcomponent/blob/6896ed2dec78919f848e874c25c3136abc61f74c/charts/zac/README.md

Looking at `db` for instance, the object itself is now described and the `db.host` key. The other ones are lost now as individual entries. 
created_at: 2025-05-07T08:24:39Z
author_association: MEMBER
diff_hunk: @@ -48,57 +48,42 @@ The Github workflow will perform helm-linting and will bump the version if neede
 
 | Key | Type | Default | Description |
 |-----|------|---------|-------------|
+| additionalAllowedFileTypes | string | `nil` | An optional list of additional file extensions that can be uploaded |
 | affinity | object | `{}` | set affinity parameters |
-| auth.clientId | string | `""` | Client ID and secret as defined in the realm |
-| auth.realm | string | `""` |  |
-| auth.secret | string | `""` |  |
-| auth.server | string | `""` |  |
-| autoscaling.enabled | bool | `false` |  |
-| autoscaling.maxReplicas | int | `100` |  |
-| autoscaling.minReplicas | int | `1` |  |
-| autoscaling.targetCPUUtilizationPercentage | int | `80` |  |
-| backendConfig.enabled | bool | `false` |  |
-| bagApi.apiKey | string | `""` |  |
-| bagApi.url | string | `""` |  |
+| auth | object | `{"clientId":"","realm":"","secret":"","server":""}` | Configuration for the Keycloak OpenID Connect integration |
+| auth.clientId | string | `""` | Client ID and secret as defined in the Keycloak ZAC realm |
+| autoscaling | object | `{"enabled":false,"maxReplicas":100,"minReplicas":1,"targetCPUUtilizationPercentage":80}` | autoscaling parameters |
path: charts/zac/README.md
user: login: edgarvonk
body: Me too. Looks good to me! You know that the descriptions are on the right hand side? You need to scroll horizontally.
created_at: 2025-05-07T08:49:17Z
author_association: MEMBER
diff_hunk: @@ -48,57 +48,42 @@ The Github workflow will perform helm-linting and will bump the version if neede
 
 | Key | Type | Default | Description |
 |-----|------|---------|-------------|
+| additionalAllowedFileTypes | string | `nil` | An optional list of additional file extensions that can be uploaded |
 | affinity | object | `{}` | set affinity parameters |
-| auth.clientId | string | `""` | Client ID and secret as defined in the realm |
-| auth.realm | string | `""` |  |
-| auth.secret | string | `""` |  |
-| auth.server | string | `""` |  |
-| autoscaling.enabled | bool | `false` |  |
-| autoscaling.maxReplicas | int | `100` |  |
-| autoscaling.minReplicas | int | `1` |  |
-| autoscaling.targetCPUUtilizationPercentage | int | `80` |  |
-| backendConfig.enabled | bool | `false` |  |
-| bagApi.apiKey | string | `""` |  |
-| bagApi.url | string | `""` |  |
+| auth | object | `{"clientId":"","realm":"","secret":"","server":""}` | Configuration for the Keycloak OpenID Connect integration |
+| auth.clientId | string | `""` | Client ID and secret as defined in the Keycloak ZAC realm |
+| autoscaling | object | `{"enabled":false,"maxReplicas":100,"minReplicas":1,"targetCPUUtilizationPercentage":80}` | autoscaling parameters |
path: charts/zac/README.md
user: login: edgarvonk
body: Ah, I see what you mean now..

for example 

```
| db.name | string | `""` |  |
| db.password | string | `""` |  |
| db.user | string | `""` |  |
```

are now no longer part of the README at all. This is strange indeed. They don't have field descriptions but that was already the case. I'll look a bit further into this.
created_at: 2025-05-07T08:52:36Z
author_association: MEMBER
diff_hunk: @@ -48,57 +48,42 @@ The Github workflow will perform helm-linting and will bump the version if neede
 
 | Key | Type | Default | Description |
 |-----|------|---------|-------------|
+| additionalAllowedFileTypes | string | `nil` | An optional list of additional file extensions that can be uploaded |
 | affinity | object | `{}` | set affinity parameters |
-| auth.clientId | string | `""` | Client ID and secret as defined in the realm |
-| auth.realm | string | `""` |  |
-| auth.secret | string | `""` |  |
-| auth.server | string | `""` |  |
-| autoscaling.enabled | bool | `false` |  |
-| autoscaling.maxReplicas | int | `100` |  |
-| autoscaling.minReplicas | int | `1` |  |
-| autoscaling.targetCPUUtilizationPercentage | int | `80` |  |
-| backendConfig.enabled | bool | `false` |  |
-| bagApi.apiKey | string | `""` |  |
-| bagApi.url | string | `""` |  |
+| auth | object | `{"clientId":"","realm":"","secret":"","server":""}` | Configuration for the Keycloak OpenID Connect integration |
+| auth.clientId | string | `""` | Client ID and secret as defined in the Keycloak ZAC realm |
+| autoscaling | object | `{"enabled":false,"maxReplicas":100,"minReplicas":1,"targetCPUUtilizationPercentage":80}` | autoscaling parameters |
path: charts/zac/README.md
user: login: niorg
body: But where's `db.name`, `db.user` and `db.password` ? That's now only listed in the object which is a pretty useless entry to me
created_at: 2025-05-07T08:54:05Z
author_association: MEMBER
diff_hunk: @@ -48,57 +48,42 @@ The Github workflow will perform helm-linting and will bump the version if neede
 
 | Key | Type | Default | Description |
 |-----|------|---------|-------------|
+| additionalAllowedFileTypes | string | `nil` | An optional list of additional file extensions that can be uploaded |
 | affinity | object | `{}` | set affinity parameters |
-| auth.clientId | string | `""` | Client ID and secret as defined in the realm |
-| auth.realm | string | `""` |  |
-| auth.secret | string | `""` |  |
-| auth.server | string | `""` |  |
-| autoscaling.enabled | bool | `false` |  |
-| autoscaling.maxReplicas | int | `100` |  |
-| autoscaling.minReplicas | int | `1` |  |
-| autoscaling.targetCPUUtilizationPercentage | int | `80` |  |
-| backendConfig.enabled | bool | `false` |  |
-| bagApi.apiKey | string | `""` |  |
-| bagApi.url | string | `""` |  |
+| auth | object | `{"clientId":"","realm":"","secret":"","server":""}` | Configuration for the Keycloak OpenID Connect integration |
+| auth.clientId | string | `""` | Client ID and secret as defined in the Keycloak ZAC realm |
+| autoscaling | object | `{"enabled":false,"maxReplicas":100,"minReplicas":1,"targetCPUUtilizationPercentage":80}` | autoscaling parameters |
path: charts/zac/README.md
user: login: edgarvonk
body: Weird because according to https://github.com/norwoodj/helm-docs this is not how it should work..
created_at: 2025-05-07T09:15:29Z
author_association: MEMBER
diff_hunk: @@ -48,57 +48,42 @@ The Github workflow will perform helm-linting and will bump the version if neede
 
 | Key | Type | Default | Description |
 |-----|------|---------|-------------|
+| additionalAllowedFileTypes | string | `nil` | An optional list of additional file extensions that can be uploaded |
 | affinity | object | `{}` | set affinity parameters |
-| auth.clientId | string | `""` | Client ID and secret as defined in the realm |
-| auth.realm | string | `""` |  |
-| auth.secret | string | `""` |  |
-| auth.server | string | `""` |  |
-| autoscaling.enabled | bool | `false` |  |
-| autoscaling.maxReplicas | int | `100` |  |
-| autoscaling.minReplicas | int | `1` |  |
-| autoscaling.targetCPUUtilizationPercentage | int | `80` |  |
-| backendConfig.enabled | bool | `false` |  |
-| bagApi.apiKey | string | `""` |  |
-| bagApi.url | string | `""` |  |
+| auth | object | `{"clientId":"","realm":"","secret":"","server":""}` | Configuration for the Keycloak OpenID Connect integration |
+| auth.clientId | string | `""` | Client ID and secret as defined in the Keycloak ZAC realm |
+| autoscaling | object | `{"enabled":false,"maxReplicas":100,"minReplicas":1,"targetCPUUtilizationPercentage":80}` | autoscaling parameters |
path: charts/zac/README.md
user: login: edgarvonk
body: ahh...

> Adding a description comment for a non-empty list or map in this way makes it so that leaf nodes underneath the described field will not be automatically added to the values table. In order to document both a non-empty list/map and a leaf node within that field, description comments must be added for both

Clear. But a bit weird if you ask me. I will update the PR so that we get all leaf nodes back in the readme even if there are no descriptions (although probably good to add descriptions for all keys anyway you could argue.).
created_at: 2025-05-07T09:48:02Z
author_association: MEMBER
diff_hunk: @@ -144,25 +150,27 @@ signaleringen:
   securityContext: {}
   tolerations: []
 
-# ZAC OpenTelemetry configuration. Only read when opentelemetry-collector is enabled
-# opentelemetry_zaakafhandelcomponent:
-#   disabled: "-true"
-#   endpoint: "http://otel-collector:4317"
+# OpenTelemetry configuration. Only read when opentelemetry-collector is enabled
+opentelemetry_zaakafhandelcomponent:
+  # -- Enables or disables the ZAC OpenTelemetry integration. Disabled by default.
+  disabled: "-true"
+  # -- OpenTelemetry collector endpoint URL
+  endpoint: ""
 
 
 # -- Maximum size (in Mega Bytes) of files that can be uploaded.
 maxFileSizeMB: 80
 
 # -- An optional list of additional file extensions that can be uploaded
-# additionalAllowedFileTypes: aaa,bbb,ccc
+additionalAllowedFileTypes:
path: charts/zac/values.yaml
user: login: edgarvonk
body: I do wonder if this change is ok. My idea here was that uncommented (optional) values are of course not documented in the generated README.md and I think they should really.. 

With uncommenting I am not 100% sure this is handled ok. Does the env var now get set or not? It should not ideally. Do you know @niorg ?

```
  {{- if .Values.additionalAllowedFileTypes }}
  ADDITIONAL_ALLOWED_FILE_TYPES: {{ .Values.additionalAllowedFileTypes }}
  {{- end}}
```

Logically the if should test: `if .Values.additionalAllowedFileTypes is not empty` so to speak..


created_at: 2025-05-07T10:09:48Z
author_association: MEMBER
diff_hunk: @@ -7,42 +7,47 @@
 # This is a YAML-formatted file.
 # Declare variables to be passed into your templates.
 
+# Please add Helm Docs field descriptions using the format `# -- blah` to leaf nodes only and not to parent nodes, except if all leaf nodes
+# have field descriptions. Otherwise, leaf nodes with field descriptions will be ignored and will not be included in the generated README.md.
+# See: https://github.com/norwoodj/helm-docs for details.
+
 # -- External URL to the zaakafhandelcomponent. (https://zaakafhandelcomponent.example.com)
 contextUrl: ""
 
-# db Configuration of database connection
+# Configuration of the ZAC database connection
 db:
-  host: ""  # -- database.internal or 1.2.3.4
+  # -- database.internal or 1.2.3.4
+  host: ""
   name: ""
   user: ""
   password: ""
 
-# auth Configuration of OpenID Connect
+# Configuration for the Keycloak OpenID Connect integration
 auth:
-  server: ""  # https://keycloak.example.com
+  server: ""
   realm: ""
-  # -- Client ID and secret as defined in the realm
+  # -- Client ID and secret as defined in the Keycloak ZAC realm
   clientId: ""
   secret: ""
 
-# configuration of the ZAC admin client in Keycloak
+# Configuration of the ZAC admin client in Keycloak
 keycloak:
   adminClient:
     # -- Keycloak ZAC admin client name
     id: ""
     # -- Keycloak ZAC admin client secret
     secret: ""
 
-# gemeente Configuration gemeente
+# -- Council specific configuration
path: charts/zac/values.yaml
user: login: niorg
body: Changed the wrong way? ;) 
```suggestion
# Council specific configuration
```
created_at: 2025-05-07T10:09:59Z
author_association: MEMBER
diff_hunk: @@ -284,7 +292,7 @@ opa:
     annotations: {}
   image:
     repository: openpolicyagent/opa
-    tag: "1.4.2-static@sha256:3c995dc8a59f6ddfd92eb7404d2f7ff9fe71cd025d9251199957a8a6afbfd76e"
+    tag: "1.4.0-static@sha256:8eb5ef478f757fabba76dfdafb58ab85667c151415b4f3689d9f05acc635d8ea"
path: charts/zac/values.yaml
user: login: niorg
body: Why the downgrade?
created_at: 2025-05-07T10:10:48Z
author_association: MEMBER
diff_hunk: @@ -284,7 +292,7 @@ opa:
     annotations: {}
   image:
     repository: openpolicyagent/opa
-    tag: "1.4.2-static@sha256:3c995dc8a59f6ddfd92eb7404d2f7ff9fe71cd025d9251199957a8a6afbfd76e"
+    tag: "1.4.0-static@sha256:8eb5ef478f757fabba76dfdafb58ab85667c151415b4f3689d9f05acc635d8ea"
path: charts/zac/values.yaml
user: login: edgarvonk
body: huh? good catch! no idea where this came from
created_at: 2025-05-07T10:17:59Z
author_association: MEMBER
diff_hunk: @@ -284,7 +292,7 @@ opa:
     annotations: {}
   image:
     repository: openpolicyagent/opa
-    tag: "1.4.2-static@sha256:3c995dc8a59f6ddfd92eb7404d2f7ff9fe71cd025d9251199957a8a6afbfd76e"
+    tag: "1.4.0-static@sha256:8eb5ef478f757fabba76dfdafb58ab85667c151415b4f3689d9f05acc635d8ea"
path: charts/zac/values.yaml
user: login: edgarvonk
body: fixed!
created_at: 2025-05-07T10:20:53Z
author_association: MEMBER
diff_hunk: @@ -144,25 +150,27 @@ signaleringen:
   securityContext: {}
   tolerations: []
 
-# ZAC OpenTelemetry configuration. Only read when opentelemetry-collector is enabled
-# opentelemetry_zaakafhandelcomponent:
-#   disabled: "-true"
-#   endpoint: "http://otel-collector:4317"
+# OpenTelemetry configuration. Only read when opentelemetry-collector is enabled
+opentelemetry_zaakafhandelcomponent:
+  # -- Enables or disables the ZAC OpenTelemetry integration. Disabled by default.
+  disabled: "-true"
+  # -- OpenTelemetry collector endpoint URL
+  endpoint: ""
 
 
 # -- Maximum size (in Mega Bytes) of files that can be uploaded.
 maxFileSizeMB: 80
 
 # -- An optional list of additional file extensions that can be uploaded
-# additionalAllowedFileTypes: aaa,bbb,ccc
+additionalAllowedFileTypes:
path: charts/zac/values.yaml
user: login: niorg
body: an empty string will evaluate as false here: https://helm.sh/docs/chart_template_guide/control_structures/
created_at: 2025-05-07T10:39:31Z
author_association: MEMBER
diff_hunk: @@ -144,25 +150,27 @@ signaleringen:
   securityContext: {}
   tolerations: []
 
-# ZAC OpenTelemetry configuration. Only read when opentelemetry-collector is enabled
-# opentelemetry_zaakafhandelcomponent:
-#   disabled: "-true"
-#   endpoint: "http://otel-collector:4317"
+# OpenTelemetry configuration. Only read when opentelemetry-collector is enabled
+opentelemetry_zaakafhandelcomponent:
+  # -- Enables or disables the ZAC OpenTelemetry integration. Disabled by default.
+  disabled: "-true"
+  # -- OpenTelemetry collector endpoint URL
+  endpoint: ""
 
 
 # -- Maximum size (in Mega Bytes) of files that can be uploaded.
 maxFileSizeMB: 80
 
 # -- An optional list of additional file extensions that can be uploaded
-# additionalAllowedFileTypes: aaa,bbb,ccc
+additionalAllowedFileTypes:
path: charts/zac/values.yaml
user: login: edgarvonk
body: ah great thanks. then this should be fine
created_at: 2025-05-07T11:03:32Z
author_association: MEMBER

PR comments:
user: login: github-actions[bot]
created_at: 2025-05-06T19:34:16Z
body: ## unit-test-results
 96 files  ±0  140 suites  ±0   1m 51s ⏱️ ±0s
573 tests ±0  573 ✅ ±0  0 💤 ±0  0 ❌ ±0 
703 runs  ±0  703 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 1072da54. ± Comparison against base commit f8aac390.

[test-results]:data:application/gzip;base64,H4sIANs3G2gC/12Oyw7CIBQFf6Vh7YLLQ8SfMVcoCbEthsKq8d8F0iq6nDnJ5GzE+WlcyXXQ59NA1uxTIxC0oM0Rkw9LFQBFlDHVWSp+0G3Nxvyrh38WRT/CoZ9+xBhjiLuJealNRfkOR7I332LjLti475kwzz4VIEAVsygFMMmNQyosGgoCy1cB+sJROXdnVpPXG6scNU4IAQAA

:recycle: This comment has been updated with latest results.
author_association: CONTRIBUTOR
user: login: github-actions[bot]
created_at: 2025-05-06T19:39:12Z
body: ## integration-test-results
 36 files  ±0   36 suites  ±0   41s ⏱️ ±0s
127 tests ±0  127 ✅ ±0  0 💤 ±0  0 ❌ ±0 
145 runs  ±0  145 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 1072da54. ± Comparison against base commit f8aac390.

[test-results]:data:application/gzip;base64,H4sIAA45G2gC/13MQQ7CIBCF4as0rF0wMIj1MmaEkhDbYiisGu8uktqiy+9/yVuZ8+OwsGsnz6eOLdmnQzZHSj7MxQjFZUqfEYT+6rZkY/7Twz9L4ntw5MefMMQY4lZinusnqg37ZVOOx+rmsLr9M2GafCpgwLWwpBCEksYRR0uGA5LSEqG/SNLO3YXt2esN6/1HoQYBAAA=

:recycle: This comment has been updated with latest results.
author_association: CONTRIBUTOR
user: login: codecov[bot]
created_at: 2025-05-06T19:39:35Z
body: ## [Codecov](https://app.codecov.io/gh/infonl/dimpact-zaakafhandelcomponent/pull/3275?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=infonl) Report
All modified and coverable lines are covered by tests :white_check_mark:
> Project coverage is 40.76%. Comparing base [(`f8aac39`)](https://app.codecov.io/gh/infonl/dimpact-zaakafhandelcomponent/commit/f8aac390ec49267bee7ccac19e33b8974c9ba7fe?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=infonl) to head [(`1072da5`)](https://app.codecov.io/gh/infonl/dimpact-zaakafhandelcomponent/commit/1072da541253cfa04dac014a57341983a7ffb2d9?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=infonl).
> Report is 2 commits behind head on main.

:white_check_mark: All tests successful. No failed tests found.

<details><summary>Additional details and impacted files</summary>


```diff
@@            Coverage Diff            @@
##               main    #3275   +/-   ##
=========================================
  Coverage     40.76%   40.76%           
  Complexity     3700     3700           
=========================================
  Files          1404     1404           
  Lines         36194    36194           
  Branches       3810     3810           
=========================================
  Hits          14755    14755           
  Misses        20602    20602           
  Partials        837      837           
```

| [Flag](https://app.codecov.io/gh/infonl/dimpact-zaakafhandelcomponent/pull/3275/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=infonl) | Coverage Δ | |
|---|---|---|
| [integrationtests](https://app.codecov.io/gh/infonl/dimpact-zaakafhandelcomponent/pull/3275/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=infonl) | `44.87% <ø> (ø)` | |
| [unittests](https://app.codecov.io/gh/infonl/dimpact-zaakafhandelcomponent/pull/3275/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=infonl) | `28.46% <ø> (ø)` | |

Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=infonl#carryforward-flags-in-the-pull-request-comment) to find out more.

</details>

[:umbrella: View full report in Codecov by Sentry](https://app.codecov.io/gh/infonl/dimpact-zaakafhandelcomponent/pull/3275?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=infonl).   
:loudspeaker: Have feedback on the report? [Share it here](https://about.codecov.io/codecov-pr-comment-feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=infonl).

<details><summary> :rocket: New features to boost your workflow: </summary>

- :package: [JS Bundle Analysis](https://docs.codecov.com/docs/javascript-bundle-analysis): Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
</details>
author_association: NONE
user: login: edgarvonk
created_at: 2025-05-07T07:23:24Z
body: Ah pff, so we have this Markdown link inspector (I think you added this @bas-info-nl ) which checks if all links in Markdown files are reachable or something? So this also means that we can not use fictional links anymore right? Which you sometimes do in documentation of course.. Like in this case.
author_association: MEMBER