Skip to content

Commit

Permalink
Merge pull request #19 from mjwloch/kubeitDefaultHosts-tenantPrefixEn…
Browse files Browse the repository at this point in the history
…abled

Moved tenantPrefixEnabled to kubeitDefaultHosts section. Fixed a related bug for non-kubeit deployments.
  • Loading branch information
mjwloch authored Nov 28, 2022
2 parents 54473a0 + 56b6359 commit 99a3bde
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion charts/platform-service/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,4 @@ apiVersion: v1
appVersion: "1.1"
description: Platform service chart
name: platform-service
version: 1.1.0
version: 1.1.1
4 changes: 2 additions & 2 deletions charts/platform-service/templates/_helpers.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -43,13 +43,13 @@ app.kubernetes.io/name: {{ include "platform-service.name" . }}
helm.sh/chart: {{ include "platform-service.chart" . }}
app.kubernetes.io/instance: {{ .Release.Name }}
version: {{ .Values.version }}
{{ if $.Values.kubeit }}
{{ if $.Values.kubeit.tenantName }}
tenant: {{ $.Values.kubeit.tenantName }}
{{- end -}}
{{- if or ( .Values.podIdentityName ) ( .Values.kubeit.tenantPodIdentityName ) }}
aadpodidbinding: {{ .Values.podIdentityName | default .Values.kubeit.tenantPodIdentityName }}
{{- end }}
{{ if $.Values.volumes -}}
state: stateful
{{- end -}}
{{- end -}}
{{- end -}}
2 changes: 1 addition & 1 deletion charts/platform-service/templates/virtualservice.yaml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
{{ $tenantPrefix := "" }}
{{- if $.Values.kubeit.tenantPrefixEnabled }}
{{- if $.Values.kubeitDefaultHosts.tenantPrefixEnabled }}
{{ $tenantPrefix = printf "-%s" .Values.kubeit.tenantName }}
{{- end }}

Expand Down
4 changes: 2 additions & 2 deletions charts/platform-service/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -83,6 +83,7 @@ defaultRouting:
enabled: true

kubeitDefaultHosts:
tenantPrefixEnabled: true
enabled: false
# hostsPrefix: # Optional - default app name

Expand Down Expand Up @@ -270,7 +271,6 @@ containerSecurityContext: {}

debugEnvoyFilters: true

kubeit:
tenantPrefixEnabled: true
#kubeit: #placeholder used for kubeit parameters passed from tenant bootstrap app of apps

initContainersDefinitions: []

0 comments on commit 99a3bde

Please sign in to comment.