devthejo
commented
1 year ago ça aurait été relayé sans cette ligne dans le chart, au contraire elle empêche l'override car ~tpl~annotations étant compilé après écrasera toujours annotations, si tu veux ajouter ce comportement par défaut (prendre les annotations globales de l'ingress) et garder la possibilité d'override il faut remplacer
~tpl~annotations: "{{ .Values.global.ingress.annotations | toJson }}"
par
~tpl~annotations: "{{ or .Values.ingress.annotations .Values.global.ingress.annotations | toJson }}"mais étant donné que .Values.ingress.annotations est définit par défaut ça ne marchera pas
depuis cette semaine on peut utiliser les valeurs du projet spécifiquement mais c'est plus complexe car tu n'aura pas forcément l'arborescence de clés définies (du genre ingress peut ne pas exister au niveau projet)
quelque chose du genre
"{{ or .Values.Parent._ProjectValues.pgweb.ingress.annotations .Values.global.ingress.annotations | toJson }}"mais en vérifiant l'existence de toute les clés de .Values.Parent._ProjectValues.pgweb.ingress.annotations pour ne pas avoir d'erreur helm
ou autre possibilité, considérer que .Values.global.ingress.annotations ne peut être entièrement optout en redéfinissant au niveau local d'un chart et du coup fusionner les deux (ce qui permet également d'écraser une valeur)
"{{ mergeOverwrite .Values.global.ingress.annotations .Values.ingress.annotations | toJson }}"
revolunet
Looks like when using pgweb kontinuous chart the
kubernetes.io/ingress.class: nginxannotation is not relayedPourtant ca a l'air relayé dans le chart kontinuous/fabrique/pg