amoghrajesh opened a new issue, #29621:
URL: https://github.com/apache/airflow/issues/29621

   ### 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: 
https://github.com/apache/airflow/pull/29270#pullrequestreview-1304890651, 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 
https://github.com/apache/airflow/pull/29270#pullrequestreview-1304890651
   
   ### How to reproduce
   
   -
   
   ### Anything else
   
   _No response_
   
   ### Are you willing to submit PR?
   
   - [X] Yes I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [X] I agree to follow this project's [Code of 
Conduct](https://github.com/apache/airflow/blob/main/CODE_OF_CONDUCT.md)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@airflow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to