Skip to content

Run UI tests selectively#15009

Merged
ashb merged 1 commit into
apache:masterfrom
astronomer:select-tests-for-new-ui
Mar 31, 2021
Merged

Run UI tests selectively#15009
ashb merged 1 commit into
apache:masterfrom
astronomer:select-tests-for-new-ui

Conversation

@ashb

@ashb ashb commented Mar 25, 2021

Copy link
Copy Markdown
Member

Closes #14957


^ 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.

@ashb ashb force-pushed the select-tests-for-new-ui branch from 1ec68de to 35b9d64 Compare March 25, 2021 16:09
@github-actions

Copy link
Copy Markdown
Contributor

The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.

@ashb ashb force-pushed the select-tests-for-new-ui branch 3 times, most recently from e21477e to a4f7daf Compare March 25, 2021 20:37
@github-actions

Copy link
Copy Markdown
Contributor

The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.

@ashb ashb force-pushed the select-tests-for-new-ui branch 2 times, most recently from d1b818a to f92590f Compare March 26, 2021 09:32
@ryanahamilton ryanahamilton force-pushed the select-tests-for-new-ui branch from f92590f to b6eb459 Compare March 30, 2021 00:48
@ryanahamilton ryanahamilton force-pushed the select-tests-for-new-ui branch 2 times, most recently from 9fd119c to 12bf58e Compare March 31, 2021 13:33
If we only change the new React UI files, we don't want to have to run
the whole python test suite!
@ashb ashb force-pushed the select-tests-for-new-ui branch from 12bf58e to d3d7b7a Compare March 31, 2021 20:13
@ashb ashb marked this pull request as ready for review March 31, 2021 20:13
@ashb ashb requested review from kaxil and potiuk as code owners March 31, 2021 20:13
@ashb

ashb commented Mar 31, 2021

Copy link
Copy Markdown
Member Author

Note that this PR will build everything as the CI scripts have changed.

@ashb ashb requested a review from ryanahamilton March 31, 2021 20:13
@ashb ashb changed the title WIP: run UI tests Run UI tests selectively Mar 31, 2021
@github-actions github-actions Bot added the full tests needed We need to run full set of tests for this PR to merge label Mar 31, 2021
@github-actions

Copy link
Copy Markdown
Contributor

The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and take the risk. If they don't merge it quickly - please rebase it to the latest master at your convenience, or amend the last commit of the PR, and push it with --force-with-lease.

@ashb ashb merged commit 7417f81 into apache:master Mar 31, 2021
@ashb ashb deleted the select-tests-for-new-ui branch March 31, 2021 22:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dev-tools full tests needed We need to run full set of tests for this PR to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Run selective CI pipeline for UI-only PRs

2 participants