Merge pull request #211 from ampls/default_udp_fix

Fix default ports for dns:// scheme [rebased]
This commit is contained in:
Hagai Barel 2025-05-06 12:41:13 +02:00 committed by GitHub
commit 0aecc93a37
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 4 additions and 5 deletions

View File

@ -1,6 +1,6 @@
apiVersion: v2 apiVersion: v2
name: coredns name: coredns
version: 1.42.0 version: 1.42.1
appVersion: 1.12.0 appVersion: 1.12.0
home: https://coredns.io home: https://coredns.io
icon: https://coredns.io/images/CoreDNS_Colour_Horizontal.png icon: https://coredns.io/images/CoreDNS_Colour_Horizontal.png
@ -19,5 +19,5 @@ maintainers:
type: application type: application
annotations: annotations:
artifacthub.io/changes: | artifacthub.io/changes: |
- kind: changed - kind: removed
description: Added Helm unit tests for CoreDNS chart components description: Removed "istcp" true for default dns:// scheme

View File

@ -99,10 +99,9 @@ Generate the list of ports automatically from the server definitions
{{- end -}} {{- end -}}
{{- end -}} {{- end -}}
{{/* If none of the zones specify scheme, default to dns:// on both tcp & udp */}} {{/* If none of the zones specify scheme, default to dns:// udp */}}
{{- if and (not (index $innerdict "istcp")) (not (index $innerdict "isudp")) -}} {{- if and (not (index $innerdict "istcp")) (not (index $innerdict "isudp")) -}}
{{- $innerdict := set $innerdict "isudp" true -}} {{- $innerdict := set $innerdict "isudp" true -}}
{{- $innerdict := set $innerdict "istcp" true -}}
{{- end -}} {{- end -}}
{{- if .nodePort -}} {{- if .nodePort -}}