From 1fb52c46f8addec2651e7118691ef0ed029d51d5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jan=20Wa=C5=9B?= <jan.was@starburstdata.com>
Date: Thu, 3 Oct 2024 10:06:35 +0200
Subject: [PATCH] Remove invalid default from

---
 charts/trino/README.md                             |  9 ++++++++-
 charts/trino/templates/deployment-coordinator.yaml |  2 ++
 charts/trino/values.yaml                           | 10 ++++++++--
 3 files changed, 18 insertions(+), 3 deletions(-)

diff --git a/charts/trino/README.md b/charts/trino/README.md
index 04a8243e..89b1ebc7 100644
--- a/charts/trino/README.md
+++ b/charts/trino/README.md
@@ -39,7 +39,14 @@ Fast distributed SQL query engine for big data analytics that helps you explore
 
   When true, only the content in `repository` is used as image reference
 * `image.pullPolicy` - string, default: `"IfNotPresent"`
-* `imagePullSecrets[0].name` - string, default: `"registry-credentials"`
+* `imagePullSecrets` - object, default: `{}`  
+
+  An optional list of references to secrets in the same namespace to use for pulling images.
+  Example:
+  ```yaml
+  imagePullSecrets:
+    - name: registry-credentials
+  ```
 * `server.workers` - int, default: `2`
 * `server.node.environment` - string, default: `"production"`
 * `server.node.dataDir` - string, default: `"/data/trino"`
diff --git a/charts/trino/templates/deployment-coordinator.yaml b/charts/trino/templates/deployment-coordinator.yaml
index 5b6cede4..bc8b7fb8 100644
--- a/charts/trino/templates/deployment-coordinator.yaml
+++ b/charts/trino/templates/deployment-coordinator.yaml
@@ -111,8 +111,10 @@ spec:
       initContainers:
       {{- tpl (toYaml .Values.initContainers.coordinator) . | nindent 6 }}
       {{- end }}
+      {{- if .Values.imagePullSecrets}}
       imagePullSecrets:
         {{- toYaml .Values.imagePullSecrets | nindent 8 }}
+      {{- end }}
       terminationGracePeriodSeconds: {{ .Values.coordinator.terminationGracePeriodSeconds }}
       containers:
         - name: {{ .Chart.Name }}-coordinator
diff --git a/charts/trino/values.yaml b/charts/trino/values.yaml
index 745d50d4..508462c4 100644
--- a/charts/trino/values.yaml
+++ b/charts/trino/values.yaml
@@ -28,8 +28,14 @@ image:
   useRepositoryAsSoleImageReference: false
   pullPolicy: IfNotPresent
 
-imagePullSecrets:
-  - name: registry-credentials
+# -- An optional list of references to secrets in the same namespace to use for pulling images.
+# @raw
+# Example:
+# ```yaml
+# imagePullSecrets:
+#   - name: registry-credentials
+# ```
+imagePullSecrets: {}
 
 server:
   workers: 2