Skip to content

Commit bec137c

Browse files
authored
MINOR: kubernetes-ingress: remove port name prometheus from service (#292)
This port was confusing as accessing the ingress controller metrics is done on the /metrics but on port http
1 parent 46f4815 commit bec137c

File tree

2 files changed

+0
-11
lines changed

2 files changed

+0
-11
lines changed

kubernetes-ingress/templates/controller-service.yaml

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -84,15 +84,6 @@ spec:
8484
nodePort: {{ .Values.controller.service.nodePorts.stat }}
8585
{{- end }}
8686
{{- end }}
87-
{{- if .Values.controller.service.enablePorts.prometheus }}
88-
- name: prometheus
89-
port: {{ .Values.controller.service.ports.prometheus }}
90-
protocol: TCP
91-
targetPort: {{ .Values.controller.service.targetPorts.prometheus }}
92-
{{- if .Values.controller.service.nodePorts.prometheus }}
93-
nodePort: {{ .Values.controller.service.nodePorts.prometheus }}
94-
{{- end }}
95-
{{- end }}
9687
{{- range .Values.controller.service.tcpPorts }}
9788
- name: {{ .name | trunc 15 | trimSuffix "-" }}
9889
port: {{ .port }}

kubernetes-ingress/values.yaml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -409,7 +409,6 @@ controller:
409409
http: 80
410410
https: 443
411411
stat: 1024
412-
prometheus: 6060
413412

414413
## The controller service ports for http, https and stat can be disabled by
415414
## setting below to false - this could be useful when only deploying haproxy
@@ -429,7 +428,6 @@ controller:
429428
https: https
430429
quic: quic
431430
stat: stat
432-
prometheus: prometheus
433431

434432
## Additional tcp ports to expose
435433
## This is especially useful for TCP services:

0 commit comments

Comments
 (0)