-
Notifications
You must be signed in to change notification settings - Fork 16.4k
Closed
Labels
area:helm-chartAirflow Helm ChartAirflow Helm Chartgood first issuekind:bugThis is a clearly a bugThis is a clearly a bug
Description
Official Helm Chart version
1.7.0 (latest released)
Apache Airflow version
2.3.0
Kubernetes Version
v1.24.x
Helm Chart configuration
No response
Docker Image customisations
No response
What happened
The helm chart is missing annotations on Deployment objects. This is not a feature per se, but is a great value add if users want to deploy with extra annotations. Currently, only podAnnotations are supported on Deployments
What you think should happen instead
The helm chart should contain an option to add Deployment annotations.
Example:
webserver:
# this is missing. This will add to the annotations of the Deployment itself
annotations: {}
# already present. This adds to the annotations of only the pod
podAnnotations: {}
webserver-deployment.yaml
kind: Deployment
apiVersion: apps/v1
metadata:
name: {{ .Release.Name }}-webserver
labels:
tier: airflow
component: webserver
release: {{ .Release.Name }}
chart: "{{ .Chart.Name }}-{{ .Chart.Version }}"
heritage: {{ .Release.Service }}
{{- with .Values.labels }}
{{ toYaml . | indent 4 }}
{{- end }}
annotations:
{{- with .Values.webserver.annotations }}
{{ toYaml . | indent 4 }}
{{- end }}
How to reproduce
No response
Anything else
No response
Are you willing to submit PR?
- Yes I am willing to submit a PR!
Code of Conduct
- I agree to follow this project's Code of Conduct
Metadata
Metadata
Assignees
Labels
area:helm-chartAirflow Helm ChartAirflow Helm Chartgood first issuekind:bugThis is a clearly a bugThis is a clearly a bug