Skip to content

Commit 0be49fa

Browse files
committed
chore(helm): refactor utility image config
1 parent c3789e1 commit 0be49fa

File tree

3 files changed

+48
-18
lines changed

3 files changed

+48
-18
lines changed

hosting/k8s/helm/templates/_helpers.tpl

Lines changed: 28 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -95,6 +95,34 @@ Get the full image name for supervisor
9595
{{- end }}
9696
{{- end }}
9797

98+
{{/*
99+
Get the full image name for webapp volumePermissions init container
100+
*/}}
101+
{{- define "trigger-v4.webapp.volumePermissions.image" -}}
102+
{{- $registry := .Values.global.imageRegistry | default .Values.webapp.volumePermissions.image.registry -}}
103+
{{- $repository := .Values.webapp.volumePermissions.image.repository -}}
104+
{{- $tag := .Values.webapp.volumePermissions.image.tag -}}
105+
{{- if $registry }}
106+
{{- printf "%s/%s:%s" $registry $repository $tag }}
107+
{{- else }}
108+
{{- printf "%s:%s" $repository $tag }}
109+
{{- end }}
110+
{{- end }}
111+
112+
{{/*
113+
Get the full image name for webapp tokenSyncer sidecar
114+
*/}}
115+
{{- define "trigger-v4.webapp.tokenSyncer.image" -}}
116+
{{- $registry := .Values.global.imageRegistry | default .Values.webapp.tokenSyncer.image.registry -}}
117+
{{- $repository := .Values.webapp.tokenSyncer.image.repository -}}
118+
{{- $tag := .Values.webapp.tokenSyncer.image.tag -}}
119+
{{- if $registry }}
120+
{{- printf "%s/%s:%s" $registry $repository $tag }}
121+
{{- else }}
122+
{{- printf "%s:%s" $repository $tag }}
123+
{{- end }}
124+
{{- end }}
125+
98126
{{/*
99127
PostgreSQL hostname (deprecated - used only for legacy DATABASE_HOST env var)
100128
*/}}

hosting/k8s/helm/templates/webapp.yaml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -67,9 +67,9 @@ spec:
6767
{{- toYaml . | nindent 8 }}
6868
{{- end }}
6969
initContainers:
70-
- name: init-shared
71-
image: "{{ .Values.webapp.initImage.registry }}/{{ .Values.webapp.initImage.repository }}:{{ .Values.webapp.initImage.tag }}"
72-
imagePullPolicy: {{ .Values.webapp.initImage.pullPolicy }}
70+
- name: volume-permissions
71+
image: {{ include "trigger-v4.webapp.volumePermissions.image" . }}
72+
imagePullPolicy: {{ .Values.webapp.volumePermissions.image.pullPolicy }}
7373
command: ['sh', '-c', 'mkdir -p /home/node/shared']
7474
securityContext:
7575
runAsUser: 1000
@@ -78,8 +78,8 @@ spec:
7878
mountPath: /home/node/shared
7979
containers:
8080
- name: token-syncer
81-
image: "{{ .Values.webapp.tokenSyncerImage.registry }}/{{ .Values.webapp.tokenSyncerImage.repository }}:{{ .Values.webapp.tokenSyncerImage.tag }}"
82-
imagePullPolicy: {{ .Values.webapp.tokenSyncerImage.pullPolicy }}
81+
image: {{ include "trigger-v4.webapp.tokenSyncer.image" . }}
82+
imagePullPolicy: {{ .Values.webapp.tokenSyncer.image.pullPolicy }}
8383
securityContext:
8484
runAsUser: 1000
8585
runAsNonRoot: true

hosting/k8s/helm/values.yaml

Lines changed: 15 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -48,19 +48,21 @@ webapp:
4848
tag: "" # Defaults to Chart.appVersion when empty
4949
pullPolicy: IfNotPresent
5050

51-
# Init container image configuration
52-
initImage:
53-
registry: docker.io
54-
repository: busybox
55-
tag: "1.35"
56-
pullPolicy: IfNotPresent
57-
58-
# Token syncer image configuration
59-
tokenSyncerImage:
60-
registry: docker.io
61-
repository: bitnamilegacy/kubectl
62-
tag: "1.28"
63-
pullPolicy: IfNotPresent
51+
# Init container for shared directory setup
52+
volumePermissions:
53+
image:
54+
registry: docker.io
55+
repository: busybox
56+
tag: "1.35"
57+
pullPolicy: IfNotPresent
58+
59+
# Sidecar for token syncing
60+
tokenSyncer:
61+
image:
62+
registry: docker.io
63+
repository: bitnamilegacy/kubectl
64+
tag: "1.28"
65+
pullPolicy: IfNotPresent
6466

6567
# Origin configuration
6668
appOrigin: "http://localhost:3040"

0 commit comments

Comments
 (0)