jscheffl commented on PR #45530: URL: https://github.com/apache/airflow/pull/45530#issuecomment-2585344164
> Nice catch! One edge case that I thought about though* - what if the corresponding setup task hasn't finished running yet? (if such exists ofc) For example:  If you set the DAG run to `failed` before the c luster was created, the `delete_cluster` should be skipped. > > * Maybe the current architecture already handles, but it's worth checking before merging this PR. Oh, yeah. Tested this explicitly. And actually the Scheduler had this feature before (added this to docs as well, was also new to me): If the setup task is skipped (which is the state when you mark as failed and it was not running) then the teardown is also skipped, -- 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]
