Skip to content

Conversation

@kaxil
Copy link
Member

@kaxil kaxil commented Nov 23, 2020

We unquarantined test_cli_webserver_background in
#12501 but seems like the
test is still flaky: https://github.com/apache/airflow/runs/1443468804#step:6:2531


^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.

We unquarantined test_cli_webserver_background in
apache#12501 but seems like the
test is still flaky: https://github.com/apache/airflow/runs/1443468804#step:6:2531
@kaxil kaxil requested a review from mik-laj November 23, 2020 18:07
@boring-cyborg boring-cyborg bot added area:CLI area:webserver Webserver related Issues labels Nov 23, 2020
@kaxil kaxil requested review from ashb and potiuk November 23, 2020 18:07
@github-actions
Copy link

The PR should be OK to be merged with just subset of tests as it does not modify Core of Airflow. The committers might merge it or can add a label 'full tests needed' and re-run it to run all tests if they see it is needed!

@github-actions github-actions bot added the okay to merge It's ok to merge this PR as it does not require more tests label Nov 23, 2020
@kaxil kaxil merged commit c33d86f into apache:master Nov 23, 2020
@kaxil kaxil deleted the quarantine-wb-back branch November 23, 2020 18:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:CLI area:webserver Webserver related Issues okay to merge It's ok to merge this PR as it does not require more tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants