[ https://issues.apache.org/jira/browse/AIRFLOW-949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15924840#comment-15924840 ]
Bolke de Bruin commented on AIRFLOW-949: ---------------------------------------- [~saguziel] [~aoen] > kill_process_tree does not kill the root process > ------------------------------------------------ > > Key: AIRFLOW-949 > URL: https://issues.apache.org/jira/browse/AIRFLOW-949 > Project: Apache Airflow > Issue Type: Bug > Components: utils > Affects Versions: 1.8.0rc4 > Reporter: Erik Cederstrand > Labels: patch > Attachments: helpers.patch > > > The kill_process_tree() function in airflow/utils/helper.py does not attempt > to kill the root process. Since there's also a kill_descendant_processes() > function, I assume that was the intent. > Also, according to the comments, the intent is to send first SIGTERM, and > then SIGKILL, to decendant processes. But in fact, SIGTERM is sent twice. > The attached patch fixes both problems. > This was found while investigating why the airflow worker would not kill > certain jobs that had crashed. -- This message was sent by Atlassian JIRA (v6.3.15#6346)