Skip to content

Commit

Permalink
[unbound] don't use named ports
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
vssldmtrv committed Dec 23, 2024
1 parent 197c1c7 commit 96bac91
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 11 deletions.
9 changes: 0 additions & 9 deletions system/unbound/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
2 changes: 0 additions & 2 deletions system/unbound/templates/service.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -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 }}

0 comments on commit 96bac91

Please sign in to comment.