jedcunningham commented on code in PR #38524:
URL: https://github.com/apache/airflow/pull/38524#discussion_r1569351206
##########
chart/values.schema.json:
##########
@@ -521,7 +521,9 @@
"LocalKubernetesExecutor",
"CeleryExecutor",
"KubernetesExecutor",
- "CeleryKubernetesExecutor"
+ "CeleryKubernetesExecutor",
+
"airflow.providers.amazon.aws.executors.batch.AwsBatchExecutor",
+ "airflow.providers.amazon.aws.executors.ecs.AwsEcsExecutor"
Review Comment:
I think that is okay though, it's an experimental executor so expecting
folks to be on a) the latest helm chart and b) the latest provider seems
reasonable.
We don't know the provider version in the chart, so we can't really check at
this level if things will work, we just have to deploy it. Should be a pretty
obvious error message when things start crashlooping anyways :)
--
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]
