o-nikolas commented on code in PR #38524:
URL: https://github.com/apache/airflow/pull/38524#discussion_r1568039685
##########
chart/values.schema.json:
##########
@@ -521,7 +521,8 @@
"LocalKubernetesExecutor",
"CeleryExecutor",
"KubernetesExecutor",
- "CeleryKubernetesExecutor"
+ "CeleryKubernetesExecutor",
+
"airflow.providers.amazon.aws.executors.ecs.ecs_executor.AwsEcsExecutor"
Review Comment:
I think a reasonable fix to unblock this would be to import the
`AwsEcsExecutor` class inside the `__init__.py` module of the `ecs` directory.
So that you could import it as:
`airflow.providers.amazon.aws.executors.ecs.AwsEcsExecutor`
This is under 63 characters (but above 50, but I'm confused why we're
limiting it to 50 since the documented limit for labels seems to be 63 maybe
@jedcunningham and @LipuFei can comment on that)
--
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]
