charts/fabrique/pgweb: missing ingress annotations

[revolunet]

Looks like when using pgweb kontinuous chart the kubernetes.io/ingress.class: nginx annotation is not relayed

Pourtant ca a l'air relayé dans le chart kontinuous/fabrique/pg

[devthejo]

ç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]

Ok ce qui m'étonne c'est qu'on a la meme ligne dans maildev et là bas j'ai bien les annotations 🤔

[devthejo]

Ok ce qui m'étonne c'est qu'on a la meme ligne dans maildev et là bas j'ai bien les annotations 🤔

où ça là bas ?

[revolunet]

En utilisant le helm-chart maildev l'annotation est bien placée 🤔 mais pas sur pgweb.
j'ai un exemple sur ovh-dev/template-hasura

[devthejo]

En utilisant le helm-chart maildev l'annotation est bien placée 🤔 mais pas sur pgweb. j'ai un exemple sur ovh-dev/template-hasura

it's hardcoded in maildev https://github.com/SocialGouv/helm-charts/blob/ca6291fa76b40c0f6aa5527f386fe802108bc5fc/charts/maildev/templates/ingress.yaml#L6