[stable/nginx-ingress] Update ports (#5348)

* [stable/nginx-ingress] Update ports

* Make port configurations consistent
* Use named ports to refer to target ports from services

* Bumped chart version

stable/nginx-ingress/Chart.yaml

@@ -1,5 +1,5 @@
 name: nginx-ingress
-version: 0.19.1
+version: 0.19.2
 appVersion: 0.14.0
 home: https://github.com/kubernetes/ingress-nginx
 description: An nginx Ingress controller that uses ConfigMap to store the nginx configuration.

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

@@ -53,13 +53,13 @@ spec:
     - name: "{{ $key }}-tcp"
       port: {{ $key }}
       protocol: TCP
-      targetPort: {{ $key }}
+      targetPort: "{{ $key }}-tcp"
   {{- end }}
   {{- range $key, $value := .Values.udp }}
     - name: "{{ $key }}-udp"
       port: {{ $key }}
       protocol: UDP
-      targetPort: {{ $key }}
+      targetPort: "{{ $key }}-udp"
   {{- end }}
   selector:
     app: {{ template "nginx-ingress.name" . }}

stable/nginx-ingress/templates/controller-stats-service.yaml

@@ -29,7 +29,7 @@ spec:
   ports:
     - name: stats
       port: {{ .Values.controller.stats.service.servicePort }}
-      targetPort: 18080
+      targetPort: stats
   selector:
     app: {{ template "nginx-ingress.name" . }}
     component: "{{ .Values.controller.name }}"

stable/nginx-ingress/templates/default-backend-deployment.yaml

@@ -50,7 +50,8 @@ spec:
             initialDelaySeconds: 30
             timeoutSeconds: 5
           ports:
-            - containerPort: 8080
+            - name: http
+              containerPort: 8080
               protocol: TCP
           resources:
 {{ toYaml .Values.defaultBackend.resources | indent 12 }}

stable/nginx-ingress/templates/default-backend-service.yaml

@@ -27,8 +27,10 @@ spec:
 {{ toYaml .Values.defaultBackend.service.loadBalancerSourceRanges | indent 4 }}
 {{- end }}
   ports:
-    - port: {{ .Values.defaultBackend.service.servicePort }}
-      targetPort: 8080
+    - name: http
+      port: {{ .Values.defaultBackend.service.servicePort }}
+      protocol: TCP
+      targetPort: http
   selector:
     app: {{ template "nginx-ingress.name" . }}
     component: "{{ .Values.defaultBackend.name }}"

stable/nginx-ingress/values.yaml

@@ -171,8 +171,8 @@ controller:
     healthCheckNodePort: 0
 
     targetPorts:
-      http: 80
-      https: 443
+      http: http
+      https: https
 
     type: LoadBalancer