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

   ### Official Helm Chart version
   
   1.14.0 (latest released)
   
   ### Apache Airflow version
   
   2.9.2
   
   ### Kubernetes Version
   
   EKS 1.30
   
   ### Helm Chart configuration
   
   ```
   workers:
     podAnnotations:
       "karpenter.sh/do-not-disrupt": "true"
   ```
   
   ### Docker Image customizations
   
   Yes but not relevant to this I don't think.
   
   ### What happened
   
   Configuring podAnnotations for workers results in other deployments such as 
the scheduler also getting those annotations.
   
   ### What you think should happen instead
   
   Only the worker pods should get those annotations.
   
   ### How to reproduce
   
   Configure the helm chart with worker podAnnotations such as
   
   ```
   workers:
     podAnnotations:
       "karpenter.sh/do-not-disrupt": "true"
   ```
   
   ### Anything else
   
   We added this annotation in conjunction with updating our helm chart version 
from `1.13.1` to `1.14.0` so unsure if it occurs on earlier helm chart 
versions. Having a look at the release notes though it looks like this may 
potentially be the culprit https://github.com/apache/airflow/pull/37352 since 
it looks like it overwrites `airflowPodAnnotations` with 
`workers.podAnnotations`.
   
   ### 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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to