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)