Skip to content

Commit

Permalink
fix: handling of a missing userConfig
Browse files Browse the repository at this point in the history
chore: bump the chart versions

this can happen after upgrading an E installation from a version
which does not support the app listening port to a version which does,
for apps pushed before the upgrade, and restarted after
  • Loading branch information
andreas-kupries committed Apr 4, 2023
1 parent a16d0ad commit 1969fe5
Show file tree
Hide file tree
Showing 5 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion chart/application-stateful/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ description: The helm chart for epinio applications to be deployed by
icon: https://raw.githubusercontent.com/epinio/helm-charts/main/assets/epinio.png
home: https://github.com/epinio/epinio
type: application
version: 0.1.22
version: 0.1.23
keywords:
- epinio
- paas
Expand Down
2 changes: 1 addition & 1 deletion chart/application-stateful/templates/_helpers.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -53,5 +53,5 @@ character removed.
Application listening port
*/}}
{{- define "epinio-app-listening-port" -}}
{{ .Values.userConfig.appListeningPort | default 8080 }}
{{ default 8080 (default (dict "appListeningPort" "8080") .Values.userConfig).appListeningPort }}
{{- end }}
2 changes: 1 addition & 1 deletion chart/application/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ description: The helm chart for epinio applications to be deployed by
icon: https://raw.githubusercontent.com/epinio/helm-charts/main/assets/epinio.png
home: https://github.com/epinio/epinio
type: application
version: 0.1.25
version: 0.1.26
keywords:
- epinio
- paas
Expand Down
2 changes: 1 addition & 1 deletion chart/application/templates/_helpers.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -53,5 +53,5 @@ character removed.
Application listening port
*/}}
{{- define "epinio-app-listening-port" -}}
{{ .Values.userConfig.appListeningPort | default 8080 }}
{{ default 8080 (default (dict "appListeningPort" "8080") .Values.userConfig).appListeningPort }}
{{- end }}
4 changes: 2 additions & 2 deletions chart/epinio/templates/default-app-chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ metadata:
spec:
shortDescription: Epinio standard deployment
description: Epinio standard support chart for application deployment
helmChart: https://github.com/epinio/helm-charts/releases/download/epinio-application-0.1.25/epinio-application-0.1.25.tgz
helmChart: https://github.com/epinio/helm-charts/releases/download/epinio-application-0.1.26/epinio-application-0.1.26.tgz
settings:
appListeningPort:
type: 'integer'
type: 'integer'

0 comments on commit 1969fe5

Please sign in to comment.