fix: Values yaml schema expects String for oidcConfiguration.customPa…

…rams which causes upgrades to fail #245 (#311)
open-metadata · Feb 12, 2025 · 5fee8ba · 5fee8ba
1 parent 94baf52
commit 5fee8ba
Show file tree

Hide file tree

Showing 4 changed files with 1,317 additions and 1,315 deletions.
diff --git a/charts/openmetadata/README.md b/charts/openmetadata/README.md
@@ -78,7 +78,7 @@ helm install openmetadata open-metadata/openmetadata --values <<path-to-values-f
 | openmetadata.config.authentication.oidcConfiguration.clientId.secretRef | string | `oidc-secrets` | OIDC_CLIENT_ID |
 | openmetadata.config.authentication.oidcConfiguration.clientSecret.secretKey | string | `openmetadata-oidc-client-secret` | OIDC_CLIENT_SECRET |
 | openmetadata.config.authentication.oidcConfiguration.clientSecret.secretRef | string | `oidc-secrets` | OIDC_CLIENT_SECRET |
-| openmetadata.config.authentication.oidcConfiguration.customParams | string | `Empty` | OIDC_CUSTOM_PARAMS |
+| openmetadata.config.authentication.oidcConfiguration.customParams | string | `{}` | OIDC_CUSTOM_PARAMS |
 | openmetadata.config.authentication.oidcConfiguration.disablePkce | bool | true | OIDC_DISABLE_PKCE |
 | openmetadata.config.authentication.oidcConfiguration.discoveryUri | string | `Empty` | OIDC_DISCOVERY_URI |
 | openmetadata.config.authentication.oidcConfiguration.enabled | bool | false | |

diff --git a/charts/openmetadata/templates/secrets.yaml b/charts/openmetadata/templates/secrets.yaml
@@ -248,7 +248,7 @@ data:
   OIDC_TENANT: {{ .oidcConfiguration.tenant | quote | b64enc }}
   OIDC_MAX_CLOCK_SKEW: {{ .oidcConfiguration.maxClockSkew | quote | b64enc }}
   OIDC_OM_REFRESH_TOKEN_VALIDITY: {{ .oidcConfiguration.tokenValidity | quote | b64enc }}
-  OIDC_CUSTOM_PARAMS: {{ .oidcConfiguration.customParams | quote | b64enc }}
+  OIDC_CUSTOM_PARAMS: {{ .oidcConfiguration.customParams | b64enc }}
 {{ end }}
 {{- if eq .provider "ldap" }}
   AUTHENTICATION_LDAP_HOST: {{ .ldapConfiguration.host | b64enc }}