Skip to content

Commit

Permalink
[stable/superset] Use the target port number instead of a name (helm#…
Browse files Browse the repository at this point in the history
…7857)

* [stable/superset] Use the target port number instead of a name

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 <[email protected]>

* [stable/superset] Bump version

Signed-off-by: Mikhail Zholobov <[email protected]>
  • Loading branch information
legal90 authored and k8s-ci-robot committed Feb 15, 2019
1 parent 38efafe commit 0facbbc
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion stable/superset/Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
description: Apache Superset (incubating) is a modern, enterprise-ready business intelligence web application
name: superset
version: 1.1.4
version: 1.1.5
appVersion: "0.28.1"
keywords:
- bi
Expand Down
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 0facbbc

Please sign in to comment.