From 96bac91b0514fb70266de54f1059e871208c0862 Mon Sep 17 00:00:00 2001
From: Vassil Dimitrov <vassil.dimitrov@sap.com>
Date: Mon, 23 Dec 2024 12:36:06 +0100
Subject: [PATCH] [unbound] don't use named ports

No need for the named ports. Let's simplify things.

We're mapping the unbound ports 1-to-1, i.e. service port 53
maps to unbound port 53, etc.
---
 system/unbound/templates/deployment.yaml | 9 ---------
 system/unbound/templates/service.yaml    | 2 --
 2 files changed, 11 deletions(-)

diff --git a/system/unbound/templates/deployment.yaml b/system/unbound/templates/deployment.yaml
index b60953d8f7..38dd3abb96 100644
--- a/system/unbound/templates/deployment.yaml
+++ b/system/unbound/templates/deployment.yaml
@@ -46,15 +46,6 @@ spec:
 {{ toYaml .Values.resources.unbound | indent 10 }}
         securityContext:
          privileged: true
-        ports:
-{{- range $.Values.unbound.externalPorts | required "externalPorts missing" }}
-          - name: dns-tcp-{{.}}
-            containerPort: {{.}}
-            protocol: TCP
-          - name: dns-udp-{{.}}
-            containerPort: {{.}}
-            protocol: UDP
-{{- end }}
         volumeMounts:
           - name: unbound-conf
             mountPath: /etc/unbound
diff --git a/system/unbound/templates/service.yaml b/system/unbound/templates/service.yaml
index b7849027ad..10eda868d2 100644
--- a/system/unbound/templates/service.yaml
+++ b/system/unbound/templates/service.yaml
@@ -19,11 +19,9 @@ spec:
     - name: dns-tcp-{{.}}
       protocol: TCP
       port: {{.}}
-      targetPort: dns-tcp-{{.}}
     - name: dns-udp-{{.}}
       protocol: UDP
       port: {{.}}
-      targetPort: dns-udp-{{.}}
 {{- end }}
   externalIPs:
     {{- required "A valid .Values.unbound.externalIPs required!" .Values.unbound.externalIPs | toYaml | nindent 2 }}