syedahsn commented on code in PR #31960:
URL: https://github.com/apache/airflow/pull/31960#discussion_r1232567156
##########
airflow/providers/amazon/aws/operators/eks.py:
##########
@@ -393,51 +416,41 @@ def __init__(
) -> None:
self.cluster_name = cluster_name
self.pod_execution_role_arn = pod_execution_role_arn
+ self.fargate_pod_execution_role_arn = pod_execution_role_arn
self.selectors = selectors
+ self.fargate_selectors = selectors
self.fargate_profile_name = fargate_profile_name
self.create_fargate_profile_kwargs = create_fargate_profile_kwargs or
{}
- self.wait_for_completion = wait_for_completion
+ self.wait_for_completion = False if deferrable else wait_for_completion
Review Comment:
Because as of right now, this is the only Operator that has the deferrable
property. Once deferrable mode is implemented for the other 2 operators, I'll
update `wait_for_completion` to match this one.
--
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]
