helm: Allow nodeport allocation for Ingress LB svc

This commit is to make sure that users can have option to disable node port allocation for LB service used by Ingress. https://kubernetes.io/docs/concepts/services-networking/service/#load-balancer-nodeport-allocation Signed-off-by: Tam Mach <tam.mach@cilium.io>
cilium · Jul 12, 2023 · 908ceeb · 908ceeb
1 parent 40a0fc3
commit 908ceeb
Show file tree

Hide file tree

Showing 6 changed files with 17 additions and 2 deletions.
diff --git a/Documentation/helm-values.rst b/Documentation/helm-values.rst
diff --git a/Documentation/spelling_wordlist.txt b/Documentation/spelling_wordlist.txt
@@ -120,6 +120,7 @@ Zannoni
 Zhou
 aarch
 alibabacloud
+allocateLoadBalancerNodePorts
 allocator
 allocators
 allowedConfigOverrides

diff --git a/install/kubernetes/cilium/README.md b/install/kubernetes/cilium/README.md
diff --git a/install/kubernetes/cilium/templates/cilium-ingress-service.yaml b/install/kubernetes/cilium/templates/cilium-ingress-service.yaml
@@ -28,6 +28,9 @@ spec:
   {{- if .Values.ingressController.service.loadBalancerClass }}
   loadBalancerClass: {{ .Values.ingressController.service.loadBalancerClass }}
   {{- end }}
+  {{- if (not (kindIs "invalid" .Values.ingressController.service.allocateLoadBalancerNodePorts)) }}
+  allocateLoadBalancerNodePorts: {{ .Values.ingressController.service.allocateLoadBalancerNodePorts }}
+  {{- end }}
   {{- end -}}
   {{- if .Values.ingressController.service.loadBalancerIP }}
   loadBalancerIP: {{ .Values.ingressController.service.loadBalancerIP }}

diff --git a/install/kubernetes/cilium/values.yaml b/install/kubernetes/cilium/values.yaml
diff --git a/install/kubernetes/cilium/values.yaml.tmpl b/install/kubernetes/cilium/values.yaml.tmpl
@@ -743,6 +743,9 @@ ingressController:
     loadBalancerClass: ~
     # -- Configure a specific loadBalancerIP on the shared LB service
     loadBalancerIP : ~
+    # -- Configure if node port allocation is required for LB service
+    # ref: https://kubernetes.io/docs/concepts/services-networking/service/#load-balancer-nodeport-allocation
+    allocateLoadBalancerNodePorts: ~
 
 gatewayAPI:
   # -- Enable support for Gateway API in cilium