jason810496 opened a new pull request, #75469: URL: https://github.com/apache/airflow/pull/75469
Reverts #75376 on `v3-4-test` only. #74869 stays on `main`. 3.4.0 closes today, and #75376 needs #75387 alongside it to be correct. Getting #75387 in needs more bandwidth than is left, so it is safer to take #75376 out of the release than to ship half the pair. The regression it leaves behind, from the review in https://github.com/apache/airflow/pull/75387#pullrequestreview-5627121884: on `v3-4-test` the Python task runner arms SIGALRM from `task.execution_timeout`, while the supervisor arms its deadline from `TIRunContext.execution_timeout`. When the server value is the smaller one, for example a different `default_task_execution_timeout`, a Dag file changed after serialization, `pre_execute`, or a mapped `expand(execution_timeout=...)`, the supervisor sends SIGTERM before the runner raises `AirflowTaskTimeout`, so `on_kill`, the retry policy, callbacks and listeners are all skipped. `git revert` applies with no conflicts, and all 29 files land byte-identical to their state before #75376. `SetExecutionTimeout` from #73806 is untouched, so Python tasks keep the supervisor backstop they had before. Draft until the 3.4.0 call is final. --- ##### Was generative AI tooling used to co-author this PR? - [X] Yes, with help of Claude Code Opus 5 following [the guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions) -- 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]
