[GitHub] [airflow] vandonr-amz commented on issue #32580: `execution_timeout` seems to be ignored in `EcsRunTaskOperator` when using `deferable`

["vandonr-amz (via GitHub)" <gi...@apache.org>]

vandonr-amz commented on issue #32580:
URL: https://github.com/apache/airflow/issues/32580#issuecomment-1634580545

   Hmm that's interesting because `execution_timeout` is a property of the base operator (i.e. of all operators), which is usually handled "automatically", but I don't think that plays well with deferrable...
   Some operators have included it manually in the `defer()` call, with varying logic, some others don't consider it.
   
   I'm also confused by what's the expected behavior in this report. Setting a large `execution_timeout` is not going to make the trigger run longer than `waiter_retries`.
   
   I actually see 3 bugs being reported:
    - tasks are successful when triggers time out (probably not what we want ?)
    - we hit the trigger timeout even though it should end on max_attempts by itself 60 seconds before ?
    - `execution_timeout` is not handled properly on defer


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org