Merge pull request #217 from ampls/default_udp_fix

Default udp fix

Author: hagaibarel

charts/coredns/Chart.yaml

@@ -1,6 +1,6 @@
 apiVersion: v2
 name: coredns
-version: 1.42.3
+version: 1.42.4
 appVersion: 1.12.2
 home: https://coredns.io
 icon: https://coredns.io/images/CoreDNS_Colour_Horizontal.png
@@ -19,5 +19,5 @@ maintainers:
 type: application
 annotations:
   artifacthub.io/changes: |
-    - kind: changed
-      description: Upgrade CoreDNS to 1.12.2
+    - kind: fixed
+      description: Fix service port removal if `scheme` is not defined, and enable use_tcp for default configuration.

charts/coredns/templates/_helpers.tpl

@@ -86,7 +86,7 @@ Generate the list of ports automatically from the server definitions
         TCP: tls://, grpc://, https://
         */}}
         {{- range .zones -}}
-            {{- if has (default "" .scheme) (list "dns://") -}}
+            {{- if has (default "" .scheme) (list "dns://" "") -}}
                 {{/* Optionally enable tcp for this service as well */}}
                 {{- if eq (default false .use_tcp) true }}
                     {{- $innerdict := set $innerdict "istcp" true -}}
@@ -157,7 +157,7 @@ Generate the list of ports automatically from the server definitions
         TCP: tls://, grpc://, https://
         */}}
         {{- range .zones -}}
-            {{- if has (default "" .scheme) (list "dns://") -}}
+            {{- if has (default "" .scheme) (list "dns://" "") -}}
                 {{/* Optionally enable tcp for this service as well */}}
                 {{- if eq (default false .use_tcp) true }}
                     {{- $innerdict := set $innerdict "istcp" true -}}

charts/coredns/values.yaml

@@ -107,6 +107,7 @@ securityContext:
 servers:
 - zones:
   - zone: .
+    use_tcp: true
   port: 53
   # -- expose the service on a different port
   # servicePort: 5353