-
Notifications
You must be signed in to change notification settings - Fork 16.4k
Closed
Labels
area:helm-chartAirflow Helm ChartAirflow Helm Chartkind:bugThis is a clearly a bugThis is a clearly a bugneeds-triagelabel for new issues that we didn't triage yetlabel for new issues that we didn't triage yet
Description
Official Helm Chart version
1.18.0 (latest released)
Apache Airflow version
2.11
Kubernetes Version
1.29
Helm Chart configuration
The Airflow Helm Chart provides robust schema validation, however this validation currently only allows Airflow officially supported Executors to be specified in the executor values field. See the schema here for details.
Docker Image customizations
No response
What happened
No response
What you think should happen instead
Custom executors should be allowed without the need to specify the helm --skip-schema-validation flag. One solution could be to allow any name that includes the word Executor at the end. This would loosens this validation significantly though I'm not sure how necessary this validation is to begin with. Very much open to other better solutions if anyone has them.
How to reproduce
Specifying a custom executor
executor: my.org.MyCustomExecutorresults in the following error when running helm template command:
- executor: Does not match pattern '^(LocalExecutor|LocalKubernetesExecutor|CeleryExecutor|KubernetesExecutor|CeleryKubernetesExecutor|airflow.providers.edge3.executors.EdgeExecutor|airflow.providers.amazon.aws.executors.batch.AwsBatchExecutor|airflow.providers.amazon.aws.executors.ecs.AwsEcsExecutor)(,(LocalExecutor|LocalKubernetesExecutor|CeleryExecutor|KubernetesExecutor|CeleryKubernetesExecutor|airflow.providers.edge3.executors.EdgeExecutor|airflow.providers.amazon.aws.executors.batch.AwsBatchExecutor|airflow.providers.amazon.aws.executors.ecs.AwsEcsExecutor))*$'
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 Chartkind:bugThis is a clearly a bugThis is a clearly a bugneeds-triagelabel for new issues that we didn't triage yetlabel for new issues that we didn't triage yet