This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 5399f91 Removes coverage from kubernetes tests (#16794)
5399f91 is described below
commit 5399f9124a4e75c7bb89e47c267d89b5280060ad
Author: Jarek Potiuk <[email protected]>
AuthorDate: Sat Jul 3 17:57:22 2021 +0200
Removes coverage from kubernetes tests (#16794)
The coverage generated by parallel runs of K8S tests cause often
failures of tests because temporary .coverage files were generated
in airflow sources. However the coverage of those tests was actually
wrong - it did not check the coverage of Airflow code (it is
running inside K8S in scheduler/workers/webserver pods).
This is quite a bit complex task captured in #16793 but for now we
should simply disable the coverage for those tests.
---
.github/workflows/ci.yml | 18 ++----------------
scripts/ci/kubernetes/ci_run_kubernetes_tests.sh | 5 +----
2 files changed, 3 insertions(+), 20 deletions(-)
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 39fd877..701e916 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -1036,16 +1036,9 @@ ${{ hashFiles('.pre-commit-config.yaml') }}"
if: failure()
with:
name: >
- kind-logs-
+ kind-logs-${{matrix.executor}}
path: /tmp/kind_logs_*
retention-days: 7
- - name: "Upload artifact for coverage"
- uses: actions/upload-artifact@v2
- with:
- name: >
- coverage-k8s-
- path: "./files/coverage*.xml"
- retention-days: 7
tests-helm-executor-upgrade:
timeout-minutes: 50
@@ -1107,16 +1100,9 @@ ${{ hashFiles('.pre-commit-config.yaml') }}"
if: failure()
with:
name: >
- kind-logs-
+ kind-logs-KubernetesExecutor
path: /tmp/kind_logs_*
retention-days: 7
- - name: "Upload artifact for coverage"
- uses: actions/upload-artifact@v2
- with:
- name: >
- coverage-k8s-
- path: "./files/coverage*.xml"
- retention-days: 7
push-prod-images-to-github-registry:
permissions:
diff --git a/scripts/ci/kubernetes/ci_run_kubernetes_tests.sh
b/scripts/ci/kubernetes/ci_run_kubernetes_tests.sh
index 6a03fb39..6cab719 100755
--- a/scripts/ci/kubernetes/ci_run_kubernetes_tests.sh
+++ b/scripts/ci/kubernetes/ci_run_kubernetes_tests.sh
@@ -62,9 +62,6 @@ function parse_tests_to_run() {
"--verbosity=1"
"--strict-markers"
"--durations=100"
- "--cov=airflow/"
- "--cov-config=.coveragerc"
-
"--cov-report=xml:files/coverage-${KIND_CLUSTER_NAME}-${HOST_PYTHON_VERSION}-${EXECUTOR}.xml"
"--color=yes"
"--maxfail=50"
"--pythonwarnings=ignore::DeprecationWarning"
@@ -92,7 +89,7 @@ function create_virtualenv() {
pip install --upgrade "pip==${AIRFLOW_PIP_VERSION}"
"wheel==${WHEEL_VERSION}"
- pip install pytest freezegun pytest-cov \
+ pip install pytest freezegun \
--constraint
"https://raw.githubusercontent.com/${CONSTRAINTS_GITHUB_REPOSITORY}/${DEFAULT_CONSTRAINTS_BRANCH}/constraints-${HOST_PYTHON_VERSION}.txt";
pip install -e ".[cncf.kubernetes,postgres]" \
