Skip to content
This repository was archived by the owner on Feb 22, 2022. It is now read-only.

Commit

Permalink
[stable/superset] Use the target port number instead of a name
Browse files Browse the repository at this point in the history
Older (but still actual) versions of some Network Policy providers don't support named ports as
a value of `targetPort` field in the Service spec.
Changing it to the number shouldn't break the UX for existing installations.

Signed-off-by: Mikhail Zholobov <legal90@gmail.com>
  • Loading branch information
legal90 committed Nov 19, 2018
1 parent bd50880 commit 2e69eab
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion stable/superset/templates/svc.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ spec:
ports:
- name: http
port: {{ .Values.service.port }}
targetPort: http
targetPort: 8088
protocol: TCP
{{ if (and (eq .Values.service.type "NodePort") (not (empty .Values.service.nodePort))) }}
nodePort: {{.Values.service.nodePort}}
Expand Down

0 comments on commit 2e69eab

Please sign in to comment.