SparebankenVest / public-helm-charts

Public Helm charts provided by Sparebanken Vest
https://charts.spvapi.no
28 stars 51 forks source link

Env Injector does not have Pod Annotations support #96

Closed ok-night closed 1 year ago

ok-night commented 1 year ago

Env Injector deployment does not have Pod Annotations support like the Controller deployment has. We would like to add annotations for these as well in our organisation.

The change is very simple and I have it prepared but I seem to not have rights to push to this repo to create a PR.

      annotations:
        checksum/config: {{ include (print $.Template.BasePath "/env-injector-apiservice.yaml") . | sha256sum }}
        {{- if .Values.env_injector.podAnnotations }}
        {{ toYaml .Values.env_injector.podAnnotations | nindent 8 }}
        {{- end }}

And corresponding inputs to values.yaml:

  # -- Additional pods labels
  podLabels: {}

  # -- Additional pod annotations
  podAnnotations: {}

Could someone help me? :)

181192 commented 1 year ago

Thanks @ok-night, added in chart version 2.3.4 :relaxed: