fix: issues with externalListener annotations

getditto · Jan 10, 2025 · 4ab7f22 · 4ab7f22
1 parent 5e4d7db
commit 4ab7f22
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 3 deletions.
diff --git a/charts/kafka/Chart.yaml b/charts/kafka/Chart.yaml
@@ -1,7 +1,7 @@
 apiVersion: v2
 description: Setup a Kafka cluster using Strimzi operator
 name: kafka
-version: 0.3.4
+version: 0.3.5
 type: application
 dependencies: []
 keywords:

diff --git a/charts/kafka/templates/kafka.yaml b/charts/kafka/templates/kafka.yaml
@@ -70,12 +70,12 @@ spec:
             {{- range .Values.strimzi.alternateNames }}
               - {{ tpl . $ | quote }}
             {{- end }}
-            annotations: {{ .Values.strimzi.externalListener.bootstrapAnnotations }}
+            annotations: {{ toYaml .Values.strimzi.externalListener.bootstrapAnnotations | nindent 14 }}
           brokers:
             {{- range $i, $e := until (int .Values.strimzi.replicas) }}
             - broker: {{ $e }}
               host: {{ tpl $.Values.strimzi.externalListener.brokerNameTemplate (merge (deepCopy $) (dict "brokerId" $e)) | quote }}
-              annotations: {{ .Values.strimzi.externalListener.brokerAnnotations }}
+              annotations: {{ toYaml $.Values.strimzi.externalListener.brokerAnnotations | nindent 16 }}
             {{- end }}
 {{- end }}
     readinessProbe:

diff --git a/charts/kafka/values.yaml b/charts/kafka/values.yaml
@@ -33,6 +33,18 @@ strimzi:
 
   entityOperatorLabels: {}
 
+  externalListener:
+    enabled: false
+    # ingress class name
+    ingressClass: <ingress-class>
+    bootstrapHost: <cluster-name>-bootstrap.<domain>
+    bootstrapAnnotations: 
+      external-dns.alpha.kubernetes.io/hostname: kafka-broker-1.mydomain.com.
+    brokerNameTemplate: "<cluster-name>-broker-{{.brokerId}}.<domain>"
+    brokerAnnotations:
+      external-dns.alpha.kubernetes.io/hostname: kafka-broker-1.mydomain.com.
+
+
 # enable kraft mode for dual use Kafka nodes in a common node pool
 kraft:
   enabled: false