Skip to content

Fix adding annotations for dag persistence PVC #29621

@amoghrajesh

Description

@amoghrajesh

Official Helm Chart version

1.8.0 (latest released)

Apache Airflow version

2.5.0

Kubernetes Version

v1.25.4

Helm Chart configuration

The dags persistence section doesn't have a default value for annotations and the usage looks like:

  annotations:
    {{- if .Values.dags.persistence.annotations}}
    {{- toYaml .Values.dags.persistence.annotations | nindent 4 }}
    {{- end }}

Docker Image customizations

No response

What happened

As per the review comments here: #29270 (review), due to this design, the upgrades might suffer. Fix them to be helm upgrade friendly

What you think should happen instead

The design should be written in an helm upgrade friendly way, refer to this suggestion #29270 (review)

How to reproduce

Anything else

No response

Are you willing to submit PR?

  • Yes I am willing to submit a PR!

Code of Conduct

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions