Skip to content

Commit

Permalink
Fix default ports for dns:// scheme
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
k0da committed Apr 14, 2021
1 parent c1edbbb commit af8fcfc
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions stable/coredns/templates/_helpers.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -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 */}}
Expand Down

0 comments on commit af8fcfc

Please sign in to comment.