kaxil commented on issue #70949: URL: https://github.com/apache/airflow/issues/70949#issuecomment-5165644144
The double-firing this describes is already handled: `_reset_signals()` at the top of `_fork_main` resets SIGINT/SIGTERM/SIGUSR2 to SIG_DFL in the child, so inherited handlers can't fire twice. The child receiving group signals directly after that is intentional -- it's how the task subprocess gets SIGTERM on container stop when dumb-init runs with `DUMB_INIT_SETSID=1` (the official image default). The TODO comment itself is stale; #65738 removes it and adds the reviewed version of the process-group change (`os.setpgid(0, 0)` scoped to task execution, paired with `os.killpg`). Closing as not planned. -- 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]
