Merge pull request #214 from ampls/default_udp_fix

Fix default ports for dns:// scheme of coredns.containerPorts definition

Author: hagaibarel

charts/coredns/Chart.yaml

@@ -1,6 +1,6 @@
 apiVersion: v2
 name: coredns
-version: 1.42.1
+version: 1.42.2
 appVersion: 1.12.0
 home: https://coredns.io
 icon: https://coredns.io/images/CoreDNS_Colour_Horizontal.png
@@ -20,4 +20,4 @@ type: application
 annotations:
   artifacthub.io/changes: |
     - kind: removed
-      description: Removed "istcp" true for default dns:// scheme
+      description: Removed "istcp" true for default dns:// scheme in "coredns.containerPorts" definition.

charts/coredns/templates/_helpers.tpl

@@ -170,10 +170,9 @@ Generate the list of ports automatically from the server definitions
             {{- 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")) -}}
             {{- $innerdict := set $innerdict "isudp" true -}}
-            {{- $innerdict := set $innerdict "istcp" true -}}
         {{- end -}}
 
         {{- if .hostPort -}}