From 2e9dd4e2b999492e572c0ac1a09d987ff369b627 Mon Sep 17 00:00:00 2001 From: NickLarsenNZ Date: Fri, 26 May 2023 00:28:26 +0200 Subject: [PATCH 1/2] fix(helm): Use correct default hostnames for postgres and redis (#24218) --- helm/superset/Chart.yaml | 2 +- helm/superset/values.yaml | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/helm/superset/Chart.yaml b/helm/superset/Chart.yaml index c82802ae13281..81893c32eb810 100644 --- a/helm/superset/Chart.yaml +++ b/helm/superset/Chart.yaml @@ -29,7 +29,7 @@ maintainers: - name: craig-rueda email: craig@craigrueda.com url: https://github.com/craig-rueda -version: 0.10.0 +version: 0.10.1 dependencies: - name: postgresql version: 12.1.6 diff --git a/helm/superset/values.yaml b/helm/superset/values.yaml index d7ff32308d6d3..d8336e78d1d47 100644 --- a/helm/superset/values.yaml +++ b/helm/superset/values.yaml @@ -248,11 +248,11 @@ supersetNode: - ". {{ .Values.configMountPath }}/superset_bootstrap.sh; /usr/bin/run-server.sh" connections: # -- Change in case of bringing your own redis and then also set redis.enabled:false - redis_host: '{{ template "superset.fullname" . }}-redis-headless' + redis_host: '{{ .Release.Name }}-redis-headless' # redis_password: superset redis_port: "6379" # You need to change below configuration incase bringing own PostgresSQL instance and also set postgresql.enabled:false - db_host: '{{ template "superset.fullname" . }}-postgresql' + db_host: '{{ .Release.Name }}-postgresql' db_port: "5432" db_user: superset db_pass: superset From 68fbe6282a955c57f73957642ca957dc16bd4ff3 Mon Sep 17 00:00:00 2001 From: NickLarsenNZ Date: Fri, 26 May 2023 11:41:35 +0200 Subject: [PATCH 2/2] docs(helm): update chart readme via helm-docs --- helm/superset/README.md | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/helm/superset/README.md b/helm/superset/README.md index a80ea62755cf2..eaa5c857a9e35 100644 --- a/helm/superset/README.md +++ b/helm/superset/README.md @@ -23,7 +23,7 @@ NOTE: This file is generated by helm-docs: https://github.com/norwoodj/helm-docs # superset -![Version: 0.10.0](https://img.shields.io/badge/Version-0.10.0-informational?style=flat-square) +![Version: 0.10.1](https://img.shields.io/badge/Version-0.10.1-informational?style=flat-square) Apache Superset is a modern, enterprise-ready business intelligence web application @@ -170,12 +170,12 @@ helm install my-superset superset/superset | supersetCeleryFlower.topologySpreadConstraints | list | `[]` | TopologySpreadConstrains to be added to supersetCeleryFlower deployments | | supersetNode.affinity | object | `{}` | Affinity to be added to supersetNode deployment | | supersetNode.command | list | See `values.yaml` | Startup command | -| supersetNode.connections.db_host | string | `"{{ template \"superset.fullname\" . }}-postgresql"` | | +| supersetNode.connections.db_host | string | `"{{ .Release.Name }}-postgresql"` | | | supersetNode.connections.db_name | string | `"superset"` | | | supersetNode.connections.db_pass | string | `"superset"` | | | supersetNode.connections.db_port | string | `"5432"` | | | supersetNode.connections.db_user | string | `"superset"` | | -| supersetNode.connections.redis_host | string | `"{{ template \"superset.fullname\" . }}-redis-headless"` | Change in case of bringing your own redis and then also set redis.enabled:false | +| supersetNode.connections.redis_host | string | `"{{ .Release.Name }}-redis-headless"` | Change in case of bringing your own redis and then also set redis.enabled:false | | supersetNode.connections.redis_port | string | `"6379"` | | | supersetNode.containerSecurityContext | object | `{}` | | | supersetNode.deploymentAnnotations | object | `{}` | Annotations to be added to supersetNode deployment |