airflow icon indicating copy to clipboard operation
airflow copied to clipboard

safe-to-evict annotation is not set when using KubernetesExecutor

Open csp33 opened this issue 1 year ago • 0 comments

Official Helm Chart version

1.11.0 (latest released)

Apache Airflow version

2.8.0

Kubernetes Version

1.29

Helm Chart configuration

executor: KubernetesExecutor
workers:
  safeToEvict: false

Docker Image customizations

No response

What happened

When running Airflow with KubernetesExecutor, the workers.safeToEvict configuration is not being used, as this field is only read in the workers deployment (used by CeleryExecutor).

What you think should happen instead

No response

How to reproduce

  1. Deploy Airflow using Kubernetes Executor. Set workers.safeToEvict: false in the chart
  2. Run a DAG
  3. Check worker pods annotations and see that the safe-to-evict one is missing

Anything else

Related PR: https://github.com/apache/airflow/pull/35130

Are you willing to submit PR?

  • [X] Yes I am willing to submit a PR!

Code of Conduct

csp33 avatar Feb 12 '24 08:02 csp33