Fix default ports for dns:// scheme (#2)

According to [values.yaml](https://github.com/coredns/helm/blob/master/stable/coredns/values.yaml#L130) dns:// is supposed to be udp only with optional tcp enablement through use_tcp flag. Signed-off-by: Dinar Valeev <k0da@opensuse.org>
k8gb-io · Apr 26, 2021 · 77fc3bd · 77fc3bd
1 parent 5a2cf9e
commit 77fc3bd
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/stable/coredns/Chart.yaml b/stable/coredns/Chart.yaml
@@ -1,6 +1,6 @@
 apiVersion: v2
 name: coredns
-version: 1.15.1
+version: 1.15.2
 appVersion: 1.8.0
 home: https://coredns.io
 icon: https://coredns.io/images/CoreDNS_Colour_Horizontal.png

diff --git a/stable/coredns/templates/_helpers.tpl b/stable/coredns/templates/_helpers.tpl
@@ -57,10 +57,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 -}}
 
         {{/* Write the dict back into the outer dict */}}