vandonr-amz opened a new pull request, #31838: URL: https://github.com/apache/airflow/pull/31838
fixing a bunch of problems around the ECS run task operator: - it was checking for task status even when we were not waiting for completion, which would result in a failure if the check happens fast enough because the task would still be pending - added a warning about trying to pull logs and not waiting for completion, which is just a weird thing to do, because logs might be truncated, they might not even be there yet. I'm assuming it was always undefined behavior, so I'm going ahead and not even starting the thread if wait for completion is false. - added a log when logs are not present (yet) so that if the issue persists, users have a message guiding them towards resolution rather than complete silence The system tests were not properly configured for logs too. I added the necessary configuration, a cleanup step, and moved the sensor to ecs_fargate so that we can have the normal behavior querying logs in the vanilla ecs test. -- 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]
