Allow custom database readiness image when using external database (#176)

* allow custom image for database readiness check

* bump chart version to 8.3.1

---------

Co-authored-by: Jesse Hitch <jessebot@linux.com>

Author: JeffResc

charts/coturn/Chart.yaml

@@ -4,7 +4,7 @@ type: application
 description: A Helm chart to deploy coturn
 home: "https://github.com/small-hack/coturn-chart"
 
-version: 8.4.0
+version: 8.5.0
 
 # renovate: image=coturn/coturn
 appVersion: 4.7.0

charts/coturn/README.md

@@ -1,6 +1,6 @@
 # coturn
 
-![Version: 8.4.0](https://img.shields.io/badge/Version-8.4.0-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 4.7.0](https://img.shields.io/badge/AppVersion-4.7.0-informational?style=flat-square)
+![Version: 8.5.0](https://img.shields.io/badge/Version-8.5.0-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 4.7.0](https://img.shields.io/badge/AppVersion-4.7.0-informational?style=flat-square)
 
 A Helm chart to deploy coturn
 
@@ -54,6 +54,8 @@ A Helm chart to deploy coturn
 | externalDatabase.enabled | bool | `false` | enables the use of postgresql instead of the default sqlite to use the bundled subchart, enable this, and postgresql.enable |
 | externalDatabase.existingSecret | string | `""` | name of existing Secret to use for postgresql credentials |
 | externalDatabase.hostname | string | `""` | required if externalDatabase.enabled: true and postgresql.enabled: false |
+| externalDatabase.image.repository | string | `""` | container registry and repo for database readiness docker image |
+| externalDatabase.image.tag | string | `""` | container tag for coturn database readiness docker image |
 | externalDatabase.password | string | `""` | password for database, ignored if existingSecret is passed in |
 | externalDatabase.secretKeys.database | string | `""` | key in existing Secret to use for the database name |
 | externalDatabase.secretKeys.hostname | string | `""` | key in existing Secret to use for the db's hostname |
@@ -97,4 +99,4 @@ A Helm chart to deploy coturn
 | service.type | string | `"ClusterIP"` | The type of service to deploy for routing Coturn traffic.   ClusterIP: Recommended for DaemonSet configurations. This will create a              standard Kubernetes service for Coturn within the cluster.              No external networking will be configured as the DaemonSet              will handle binding to each Node's host networking    NodePort:  Recommended for Deployment configurations. This will open              TURN ports on every node and route traffic on these ports to              the Coturn pods. You will need to make sure your cloud              provider supports the cluster config setting,              apiserver.service-node-port-range, as this range must contain              the ports defined above for the service to be created.    LoadBalancer: This was what was originally set for this chart in the                 upstream of this fork, but with no details |
 
 ----------------------------------------------
-Autogenerated from chart metadata using [helm-docs v1.11.0](https://github.com/norwoodj/helm-docs/releases/v1.11.0)
+Autogenerated from chart metadata using [helm-docs v1.14.2](https://github.com/norwoodj/helm-docs/releases/v1.14.2)

charts/coturn/templates/_helpers.tpl

@@ -53,9 +53,9 @@ Helper function to get the coturn secret containing admin coturn credentials
 
 {{- define "db.isReady.image.repository" -}}
 {{- if and .Values.externalDatabase.enabled (eq .Values.externalDatabase.type "postgresql") -}}
-postgres
+{{ .Values.externalDatabase.image.repository | default "postgres" }}
 {{- else if and .Values.externalDatabase.enabled (eq .Values.externalDatabase.type "mysql") -}}
-mysql
+{{ .Values.externalDatabase.image.repository | default "mysql" }}
 {{- else if .Values.postgresql.enabled -}}
 {{ .Values.postgresql.image.repository }}
 {{- else if .Values.mysql.enabled -}}
@@ -65,9 +65,9 @@ mysql
 
 {{- define "db.isReady.image.tag" -}}
 {{- if and .Values.externalDatabase.enabled (eq .Values.externalDatabase.type "postgresql") -}}
-15-alpine
+{{ .Values.externalDatabase.image.tag | default "15-alpine" }}
 {{- else if and .Values.externalDatabase.enabled (eq .Values.externalDatabase.type "mysql") -}}
-8.0.35
+{{ .Values.externalDatabase.image.tag | default "8.0.35" }}
 {{- else if .Values.postgresql.enabled -}}
 {{ .Values.postgresql.image.tag }}
 {{- else if .Values.mysql.enabled -}}

charts/coturn/values.yaml

@@ -76,6 +76,12 @@ externalDatabase:
     database: ""
     # -- key in existing Secret to use for the db's hostname
     hostname: ""
+  # Custom image for database readiness check
+  image:
+    # -- container registry and repo for database readiness docker image
+    repository: ""
+    # -- container tag for coturn database readiness docker image
+    tag: ""
 
 # PostgreSQL chart configuration
 # see: https://github.com/bitnami/charts/tree/main/bitnami/postgresql