Skip to content

Run KubernetesPodOperator tests on any executor#19810

Merged
dstandish merged 1 commit intoapache:mainfrom
astronomer:dont-skip-k8s-tests
Nov 29, 2021
Merged

Run KubernetesPodOperator tests on any executor#19810
dstandish merged 1 commit intoapache:mainfrom
astronomer:dont-skip-k8s-tests

Conversation

@dstandish
Copy link
Contributor

@dstandish dstandish commented Nov 24, 2021

Previously these tests were restricted to run only on KubernetesExecutor. But there is no reason we cannot run them on other executors.

@boring-cyborg boring-cyborg bot added the provider:cncf-kubernetes Kubernetes (k8s) provider related issues label Nov 24, 2021
Previously these tests were restricted to run only on KubernetesExecutor. But there is no reason
we cannot run them on other executors.
@dstandish dstandish marked this pull request as ready for review November 27, 2021 03:28
@github-actions
Copy link

The PR is likely OK to be merged with just subset of tests for default Python and Database versions without running the full matrix of tests, because it does not modify the core of Airflow. If the committers decide that the full tests matrix is needed, they will add the label 'full tests needed'. Then you should rebase to the latest main or amend the last commit of the PR, and push it with --force-with-lease.

@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 27, 2021
@dstandish dstandish merged commit 68f13f0 into apache:main Nov 29, 2021
dillonjohnson pushed a commit to dillonjohnson/airflow that referenced this pull request Dec 1, 2021
Previously these tests were restricted to run only on KubernetesExecutor. But there is no reason
we cannot run them on other executors.
@dstandish dstandish deleted the dont-skip-k8s-tests branch December 6, 2021 17:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

okay to merge It's ok to merge this PR as it does not require more tests provider:cncf-kubernetes Kubernetes (k8s) provider related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants