Skip to content

Commit 31055cd

Browse files
manicsk8s-ci-robot
authored andcommitted
Enable nginx-ingress daemonset hostports 80 443 (helm#3097)
1 parent 5092523 commit 31055cd

File tree

3 files changed

+11
-1
lines changed

3 files changed

+11
-1
lines changed

stable/nginx-ingress/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
name: nginx-ingress
2-
version: 0.8.23
2+
version: 0.8.24
33
appVersion: 0.9.0-beta.15
44
description: An nginx Ingress controller that uses ConfigMap to store the nginx configuration.
55
icon: https://upload.wikimedia.org/wikipedia/commons/thumb/c/c5/Nginx_logo.svg/500px-Nginx_logo.svg.png

stable/nginx-ingress/templates/controller-daemonset.yaml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -94,9 +94,15 @@ spec:
9494
- name: http
9595
containerPort: 80
9696
protocol: TCP
97+
{{- if .Values.controller.daemonset.useHostPorts }}
98+
hostPort: 80
99+
{{- end }}
97100
- name: https
98101
containerPort: 443
99102
protocol: TCP
103+
{{- if .Values.controller.daemonset.useHostPorts }}
104+
hostPort: 443
105+
{{- end }}
100106
{{- if .Values.controller.stats.enabled }}
101107
- name: stats
102108
containerPort: 18080

stable/nginx-ingress/values.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,10 @@ controller:
1717
# is merged
1818
hostNetwork: false
1919

20+
## Use host ports 80 and 443
21+
daemonset:
22+
useHostPort: false
23+
2024
## Required only if defaultBackend.enabled = false
2125
## Must be <namespace>/<service_name>
2226
##

0 commit comments

Comments
 (0)