vandonr-amz commented on code in PR #31881:
URL: https://github.com/apache/airflow/pull/31881#discussion_r1230188060
##########
airflow/providers/amazon/aws/operators/ecs.py:
##########
@@ -174,9 +215,21 @@ def execute(self, context: Context):
cluster_state = cluster_details.get("status")
if cluster_state == EcsClusterStates.INACTIVE:
- # In some circumstances the ECS Cluster is deleted immediately,
- # so there is no reason to wait for completion.
+ # if the cluster doesn't have capacity providers that are
associated with it,
+ # the deletion is instantaneous, and we don't need to wait for it.
Review Comment:
hmm, we could do that, but the check on the status above is already taking
care of that. We can write a different check, but the result would be the same.
--
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]
