You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "Jarek Potiuk (Jira)" <ji...@apache.org> on 2019/12/17 21:27:00 UTC

[jira] [Resolved] (AIRFLOW-5926) Fix race-condition in TestCliWebServer

     [ https://issues.apache.org/jira/browse/AIRFLOW-5926?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jarek Potiuk resolved AIRFLOW-5926.
-----------------------------------
    Resolution: Fixed

> Fix race-condition  in TestCliWebServer
> ---------------------------------------
>
>                 Key: AIRFLOW-5926
>                 URL: https://issues.apache.org/jira/browse/AIRFLOW-5926
>             Project: Apache Airflow
>          Issue Type: Bug
>          Components: cli
>    Affects Versions: 1.10.6
>            Reporter: Kamil Bregula
>            Priority: Major
>
> The current tests work as follows.
>  # The airflow command is started in a separate process, without waiting for it to finish. This separate process run the airflow process that creates the deamon process - airflow-webserver-monitor. The deamon process starts another process - gunicorn.
>  # At the same time, the tests are waiting for the monitor process to start. Checks if the pid file for the monitor exists.
>  # In the next step, the test checks that the airflow command has already ended
> *Bug* we should check the monitor process, not the command that starts it.
>  # In the next step, the test checks if the gunicorn is already running
>  # Tests sends a signal to the monitor to close processes - monitor and gunicorn.
> Summary:
> The test checks to see if there is a temporary process that starts the monitor instead of checking that the monitor is running.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)