renzepost commented on PR #49752: URL: https://github.com/apache/airflow/pull/49752#issuecomment-2838243757
> edit: This change makes sense on its own but will not close the related issue. (or can we add a test for it, teardown need to be success, 1 task in the dag need to be failed, and we are marking as success the dagrun, which will remain wrongly in failed state). The last unit test now covers this scenario with `TaskInstanceState.SUCCESS` parameter. There's a failed "normal" task, the dagrun is in failed state, the teardown will be in success state, and after marking the dagrun to success, the dagrun is indeed marked as success (and so is the failed "normal" task). -- 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]
