fix(helmfile): Conditional templating of additional annotations in selected components to unblock openDesk deployment despite a bug in Helm 3.17 (https://github.com/helm/helm/issues/30587)

This commit is contained in:
Thorsten Roßner
2025-04-23 12:18:20 +02:00
parent a9fae59b74
commit 861b84b14d
2 changed files with 28 additions and 2 deletions

View File

@@ -10,9 +10,10 @@ global:
{{ .Values.global.hosts | toYaml | nindent 4 }} {{ .Values.global.hosts | toYaml | nindent 4 }}
imagePullSecrets: imagePullSecrets:
{{ .Values.global.imagePullSecrets | toYaml | nindent 4 }} {{ .Values.global.imagePullSecrets | toYaml | nindent 4 }}
{{- if .Values.annotations.jitsiGlobal.pod }}
podAnnotations: podAnnotations:
{{ .Values.annotations.jitsiGlobal.pod | toYaml | nindent 4}} {{ .Values.annotations.jitsiGlobal.pod | toYaml | nindent 4}}
{{- end }}
containerSecurityContext: containerSecurityContext:
allowPrivilegeEscalation: false allowPrivilegeEscalation: false
enabled: true enabled: true
@@ -78,10 +79,12 @@ jitsi:
pullPolicy: {{ .Values.global.imagePullPolicy | quote }} pullPolicy: {{ .Values.global.imagePullPolicy | quote }}
ingress: ingress:
enabled: {{ .Values.ingress.enabled }} enabled: {{ .Values.ingress.enabled }}
{{- if .Values.annotations.jitsiWeb.ingress }}
annotations: annotations:
{{- with .Values.annotations.jitsiWeb.ingress }} {{- with .Values.annotations.jitsiWeb.ingress }}
{{ . | toYaml | nindent 8 }} {{ . | toYaml | nindent 8 }}
{{- end }} {{- end }}
{{- end }}
ingressClassName: {{ .Values.ingress.ingressClassName | quote }} ingressClassName: {{ .Values.ingress.ingressClassName | quote }}
hosts: hosts:
- host: "{{ .Values.global.hosts.jitsi }}.{{ .Values.global.domain }}" - host: "{{ .Values.global.hosts.jitsi }}.{{ .Values.global.domain }}"
@@ -109,8 +112,10 @@ jitsi:
type: "RuntimeDefault" type: "RuntimeDefault"
seLinuxOptions: seLinuxOptions:
{{ .Values.seLinuxOptions.jitsi | toYaml | nindent 8 }} {{ .Values.seLinuxOptions.jitsi | toYaml | nindent 8 }}
{{- if .Values.annotations.jitsiWeb.pod }}
podAnnotations: podAnnotations:
{{ .Values.annotations.jitsiWeb.pod | toYaml | nindent 6 }} {{ .Values.annotations.jitsiWeb.pod | toYaml | nindent 6 }}
{{- end }}
prosody: prosody:
image: image:
repository: "{{ coalesce .Values.repositories.image.registryOpencodeDe .Values.global.imageRegistry .Values.images.prosody.registry }}/{{ .Values.images.prosody.repository }}" repository: "{{ coalesce .Values.repositories.image.registryOpencodeDe .Values.global.imageRegistry .Values.images.prosody.registry }}/{{ .Values.images.prosody.repository }}"
@@ -160,8 +165,10 @@ jitsi:
type: "RuntimeDefault" type: "RuntimeDefault"
seLinuxOptions: seLinuxOptions:
{{ .Values.seLinuxOptions.prosody | toYaml | nindent 8 }} {{ .Values.seLinuxOptions.prosody | toYaml | nindent 8 }}
{{- if .Values.annotations.jitsiProsody.pod }}
podAnnotations: podAnnotations:
{{ .Values.annotations.jitsiProsody.pod | toYaml | nindent 6 }} {{ .Values.annotations.jitsiProsody.pod | toYaml | nindent 6 }}
{{- end }}
jicofo: jicofo:
replicaCount: {{ .Values.replicas.jicofo }} replicaCount: {{ .Values.replicas.jicofo }}
image: image:
@@ -185,8 +192,10 @@ jitsi:
type: "RuntimeDefault" type: "RuntimeDefault"
seLinuxOptions: seLinuxOptions:
{{ .Values.seLinuxOptions.jicofo | toYaml | nindent 8 }} {{ .Values.seLinuxOptions.jicofo | toYaml | nindent 8 }}
{{- if .Values.annotations.jitsiJicofo.pod }}
podAnnotations: podAnnotations:
{{ .Values.annotations.jitsiJicofo.pod | toYaml | nindent 6 }} {{ .Values.annotations.jitsiJicofo.pod | toYaml | nindent 6 }}
{{- end }}
jigasi: jigasi:
replicaCount: {{ .Values.replicas.jigasi }} replicaCount: {{ .Values.replicas.jigasi }}
enabled: {{ .Values.sip.jigasi.enabled }} enabled: {{ .Values.sip.jigasi.enabled }}
@@ -216,8 +225,10 @@ jitsi:
type: "RuntimeDefault" type: "RuntimeDefault"
seLinuxOptions: seLinuxOptions:
{{ .Values.seLinuxOptions.jigasi | toYaml | nindent 8 }} {{ .Values.seLinuxOptions.jigasi | toYaml | nindent 8 }}
{{- if .Values.annotations.jitsiJigasi.pod }}
podAnnotations: podAnnotations:
{{ .Values.annotations.jitsiJigasi.pod | toYaml | nindent 6 }} {{ .Values.annotations.jitsiJigasi.pod | toYaml | nindent 6 }}
{{- end }}
jvb: jvb:
replicaCount: {{ .Values.replicas.jvb }} replicaCount: {{ .Values.replicas.jvb }}
# The `useNodeIP` option provided by the upstream charts does not support all relevant scenarios, but since # The `useNodeIP` option provided by the upstream charts does not support all relevant scenarios, but since
@@ -234,8 +245,10 @@ jitsi:
{{ .Values.resources.jvb | toYaml | nindent 6 }} {{ .Values.resources.jvb | toYaml | nindent 6 }}
service: service:
type: {{ coalesce .Values.service.type.jitsiVideoBridge .Values.cluster.service.type | quote }} type: {{ coalesce .Values.service.type.jitsiVideoBridge .Values.cluster.service.type | quote }}
{{- if .Values.annotations.jitsiJvb.service }}
annotations: annotations:
{{ .Values.annotations.jitsiJvb.service | toYaml | nindent 8 }} {{ .Values.annotations.jitsiJvb.service | toYaml | nindent 8 }}
{{- end }}
securityContext: securityContext:
allowPrivilegeEscalation: false allowPrivilegeEscalation: false
capabilities: {} capabilities: {}
@@ -248,14 +261,18 @@ jitsi:
type: "RuntimeDefault" type: "RuntimeDefault"
seLinuxOptions: seLinuxOptions:
{{ .Values.seLinuxOptions.jvb | toYaml | nindent 8 }} {{ .Values.seLinuxOptions.jvb | toYaml | nindent 8 }}
{{- if .Values.annotations.jitsiJvb.pod }}
podAnnotations: podAnnotations:
{{ .Values.annotations.jitsiJvb.pod | toYaml | nindent 6 }} {{ .Values.annotations.jitsiJvb.pod | toYaml | nindent 6 }}
{{- end }}
metrics: metrics:
prometheusAnnotations: prometheusAnnotations:
{{ .Values.annotations.jitsiJvb.metricsPrometheus | toYaml | nindent 8 }} {{ .Values.annotations.jitsiJvb.metricsPrometheus | toYaml | nindent 8 }}
{{- if .Values.annotations.jitsiJvb.metricsGrafana }}
grafanaDashboards: grafanaDashboards:
annotations: annotations:
{{ .Values.annotations.jitsiJvb.metricsGrafana | toYaml | nindent 10 }} {{ .Values.annotations.jitsiJvb.metricsGrafana | toYaml | nindent 10 }}
{{- end }}
jibri: jibri:
replicaCount: {{ .Values.replicas.jibri }} replicaCount: {{ .Values.replicas.jibri }}
image: image:
@@ -272,16 +289,19 @@ jitsi:
# Chart does not allow to template more # Chart does not allow to template more
capabilities: capabilities:
add: ["SYS_ADMIN"] add: ["SYS_ADMIN"]
{{- if .Values.annotations.jitsiJibri.pod }}
podAnnotations: podAnnotations:
{{ .Values.annotations.jitsiJibri.pod | toYaml | nindent 6 }} {{ .Values.annotations.jitsiJibri.pod | toYaml | nindent 6 }}
{{- end }}
imagePullSecrets: imagePullSecrets:
{{- range .Values.global.imagePullSecrets }} {{- range .Values.global.imagePullSecrets }}
- name: {{ . | quote }} - name: {{ . | quote }}
{{- end }} {{- end }}
{{- if .Values.annotations.jitsi.serviceAccount }}
serviceAccount: serviceAccount:
annotations: annotations:
{{ .Values.annotations.jitsi.serviceAccount | toYaml | nindent 6 }} {{ .Values.annotations.jitsi.serviceAccount | toYaml | nindent 6 }}
{{- end }}
patchJVB: patchJVB:
configuration: configuration:
@@ -308,8 +328,10 @@ patchJVB:
repository: {{ .Values.images.jitsiPatchJVB.repository | quote }} repository: {{ .Values.images.jitsiPatchJVB.repository | quote }}
tag: {{ .Values.images.jitsiPatchJVB.tag | quote }} tag: {{ .Values.images.jitsiPatchJVB.tag | quote }}
{{- if .Values.annotations.jitsi.pod }}
podAnnotations: podAnnotations:
{{ .Values.annotations.jitsi.pod | toYaml | nindent 2 }} {{ .Values.annotations.jitsi.pod | toYaml | nindent 2 }}
{{- end }}
replicaCount: {{ .Values.replicas.jitsiKeycloakAdapter }} replicaCount: {{ .Values.replicas.jitsiKeycloakAdapter }}

View File

@@ -628,8 +628,10 @@ appsuite:
repository: {{ .Values.images.openxchangeDocumentConverter.repository | quote }} repository: {{ .Values.images.openxchangeDocumentConverter.repository | quote }}
tag: {{ .Values.images.openxchangeDocumentConverter.tag | quote }} tag: {{ .Values.images.openxchangeDocumentConverter.tag | quote }}
pullPolicy: {{ .Values.global.imagePullPolicy | quote }} pullPolicy: {{ .Values.global.imagePullPolicy | quote }}
{{- if .Values.annotations.openxchangeAppsuiteCoreDocumentconverter.pod }}
podAnnotations: podAnnotations:
{{ .Values.annotations.openxchangeAppsuiteCoreDocumentconverter.pod | toYaml | nindent 6 }} {{ .Values.annotations.openxchangeAppsuiteCoreDocumentconverter.pod | toYaml | nindent 6 }}
{{- end }}
redis: *redisConfiguration redis: *redisConfiguration
replicaCount: {{ .Values.replicas.openxchangeCoreDocumentConverter }} replicaCount: {{ .Values.replicas.openxchangeCoreDocumentConverter }}
resources: resources:
@@ -718,8 +720,10 @@ appsuite:
endpoint: "." endpoint: "."
accessKey: "." accessKey: "."
secretKey: "." secretKey: "."
{{- if .Values.annotations.openxchangeAppsuiteCoreImageconverter.pod }}
podAnnotations: podAnnotations:
{{ .Values.annotations.openxchangeAppsuiteCoreImageconverter.pod | toYaml | nindent 6 }} {{ .Values.annotations.openxchangeAppsuiteCoreImageconverter.pod | toYaml | nindent 6 }}
{{- end }}
redis: *redisConfiguration redis: *redisConfiguration
replicaCount: {{ .Values.replicas.openxchangeCoreImageConverter }} replicaCount: {{ .Values.replicas.openxchangeCoreImageConverter }}
resources: resources: