HTTP/3 disable UDP ports by default

Author: marcel2012

build/dev-env.sh

@@ -89,8 +89,6 @@ controller:
     digest:
   config:
     worker-processes: "1"
-  extraArgs:
-    enable-quic: "true"
   podLabels:
     deploy-date: "$(date +%s)"
   updateStrategy:

charts/ingress-nginx/README.md

@@ -317,7 +317,7 @@ metadata:
 | controller.configMapNamespace | string | `""` | Allows customization of the configmap / nginx-configmap namespace; defaults to $(POD_NAMESPACE) |
 | controller.containerName | string | `"controller"` | Configures the controller container name |
 | controller.containerPort | object | `{"http":80,"https":443}` | Configures the TCP ports that the nginx-controller listens on |
-| controller.containerUdpPort | object | `{"quic":443}` | Configures the UDP ports that the nginx-controller listens on |
+| controller.containerUdpPort | object | `{}` | Configures the UDP ports that the nginx-controller listens on |
 | controller.containerSecurityContext | object | `{}` | Security context for controller containers |
 | controller.customTemplate.configMapKey | string | `""` |  |
 | controller.customTemplate.configMapName | string | `""` |  |

charts/ingress-nginx/tests/controller-daemonset_test.yaml

@@ -208,18 +208,6 @@ tests:
       - equal:
           path: spec.template.spec.runtimeClassName
           value: myClass
-
-  - it: should create a DaemonSet with a default UDP container port
-    set:
-      controller.kind: DaemonSet
-    asserts:
-      - contains:
-          path: spec.template.spec.containers[0].ports
-          content:
-            name: quic
-            containerPort: 443
-            protocol: UDP
-
   - it: should create a DaemonSet with a custom UDP container port if `controller.containerUdpPort.quic` is set
     set:
       controller.kind: DaemonSet

charts/ingress-nginx/tests/controller-deployment_test.yaml

@@ -232,15 +232,6 @@ tests:
           path: spec.template.spec.runtimeClassName
           value: myClass
 
-  - it: should create a Deployment with a default UDP container port
-    asserts:
-      - contains:
-          path: spec.template.spec.containers[0].ports
-          content:
-            name: quic
-            containerPort: 443
-            protocol: UDP
-
   - it: should create a Deployment with a custom UDP container port if `controller.containerUdpPort.quic` is set
     set:
       controller.containerUdpPort.quic: 1234

charts/ingress-nginx/tests/controller-networkpolicy_test.yaml

@@ -22,16 +22,6 @@ tests:
           path: metadata.name
           value: RELEASE-NAME-ingress-nginx-controller
 
-  - it: should create a NetworkPolicy with a default UDP port
-    set:
-      controller.networkPolicy.enabled: true
-    asserts:
-      - contains:
-          path: spec.ingress[0].ports
-          content:
-            protocol: UDP
-            port: 443
-
   - it: should create a NetworkPolicy with a custom UDP port if `controller.containerUdpPort.quic` is set
     set:
       controller.networkPolicy.enabled: true

charts/ingress-nginx/values.yaml

@@ -52,8 +52,8 @@ controller:
     http: 80
     https: 443
   # -- Configures the UDP ports that the nginx-controller listens on
-  containerUdpPort:
-    quic: 443
+  containerUdpPort: []
+  # quic: 443
   # -- Global configuration passed to the ConfigMap consumed by the controller. Values may contain Helm templates.
   # Ref.: https://kubernetes.github.io/ingress-nginx/user-guide/nginx-configuration/configmap/
   config: {}