Fixed mounting of trino-password-authentication secret (#121)

* Fixed mounting of trino-password-authentication secret. * Bumped chart version. Uncommented authenticationType with empty value and added double dash to comment above it for helm-docs. Generated new readme with helm-docs.
valeriano-manassero · Jun 7, 2022 · 0b5e180 · 0b5e180
1 parent c253458
commit 0b5e180
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 6 deletions.
diff --git a/valeriano-manassero/trino/Chart.yaml b/valeriano-manassero/trino/Chart.yaml
@@ -2,7 +2,7 @@ apiVersion: v1
 appVersion: "378"
 description: High performance, distributed SQL query engine for big data
 name: trino
-version: 2.0.0
+version: 2.0.1
 home: https://trino.io
 icon: https://trino.io/assets/images/trino-logo/trino-ko_tiny-alt.svg
 sources:

diff --git a/valeriano-manassero/trino/README.md b/valeriano-manassero/trino/README.md
@@ -1,6 +1,6 @@
 # trino
 
-![Version: 2.0.0](https://img.shields.io/badge/Version-2.0.0-informational?style=flat-square) ![AppVersion: 378](https://img.shields.io/badge/AppVersion-378-informational?style=flat-square)
+![Version: 2.0.1](https://img.shields.io/badge/Version-2.0.1-informational?style=flat-square) ![AppVersion: 378](https://img.shields.io/badge/AppVersion-378-informational?style=flat-square)
 
 High performance, distributed SQL query engine for big data
 
@@ -35,6 +35,7 @@ High performance, distributed SQL query engine for big data
 | config.coordinator.replicas | int | `1` |  |
 | config.coordinator.resources | object | `{}` |  |
 | config.coordinator.tolerations | list | `[]` |  |
+| config.general.authenticationType | string | `""` | Trino supports multiple authentication types: PASSWORD, CERTIFICATE, OAUTH2, JWT, KERBEROS For more info: https://trino.io/docs/current/security/authentication-types.html |
 | config.general.http.port | int | `8080` |  |
 | config.general.httpsServer.enabled | bool | `false` |  |
 | config.general.httpsServer.keystore.key | string | `""` |  |

diff --git a/valeriano-manassero/trino/templates/deployment-coordinator.yaml b/valeriano-manassero/trino/templates/deployment-coordinator.yaml
@@ -37,7 +37,7 @@ spec:
         - name: schemas-volume
           configMap:
             name: schemas-volume-coordinator
-        {{- if .Values.config.coordinator.authenticationType }}{{- if eq .Values.config.coordinator.authenticationType "PASSWORD" }}
+        {{- if .Values.config.general.authenticationType }}{{- if eq .Values.config.general.authenticationType "PASSWORD" }}
         - name: password-volume
           secret:
             secretName: trino-password-authentication
@@ -89,7 +89,7 @@ spec:
               name: schemas-volume
             - mountPath: {{ .Values.config.general.path }}/catalog
               name: catalog-volume
-            {{- if .Values.config.coordinator.authenticationType }}{{- if eq .Values.config.coordinator.authenticationType "PASSWORD" }}
+            {{- if .Values.config.general.authenticationType }}{{- if eq .Values.config.general.authenticationType "PASSWORD" }}
             - mountPath: {{ .Values.config.general.path }}/auth
               name: password-volume
             {{- end }}{{- end }}

diff --git a/valeriano-manassero/trino/values.yaml b/valeriano-manassero/trino/values.yaml
@@ -31,9 +31,9 @@ config:
     http:
       port: 8080
     processForwarded: false
-    # Trino supports multiple authentication types: PASSWORD, CERTIFICATE, OAUTH2, JWT, KERBEROS
+    # -- Trino supports multiple authentication types: PASSWORD, CERTIFICATE, OAUTH2, JWT, KERBEROS
     # For more info: https://trino.io/docs/current/security/authentication-types.html
-    # authenticationType: "PASSWORD"
+    authenticationType: ""
     httpsServer:
       enabled: false
       port: 8443