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 ea8eabc1e7 Remove "post-tests" steps for Helm Unit tests in CI (#35437)
ea8eabc1e7 is described below
commit ea8eabc1e7fc3c5f602a42d567772567b4be05ac
Author: Jarek Potiuk <[email protected]>
AuthorDate: Sat Nov 4 14:13:07 2023 +0100
Remove "post-tests" steps for Helm Unit tests in CI (#35437)
The "Helm Unit Tests" are special case of tests - they do not
run airflow code, they are merely runnig `helm` to process the
chart templates with various combinations of parameters. As such,
they do not contribute to coverage of Airflow Python code, and they
do not generate warnings when the code is run so there is no point
in attempting to upload coverage and warnings after the tests
complete. This even generates warnings currently in Job summary:
> Unit tests Helm: other
> No files were found with the provided path: ./files/warnings-*.txt. No
> artifacts will be uploaded.
There is no real cleanup we need after those tests because we are
not running containers during those tests, so we can safely just
remove cleanup steps - not even fix-ownership that helps to deal
with files that are created with root owner during container
based jobs.
---
.github/workflows/ci.yml | 8 --------
1 file changed, 8 deletions(-)
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index d374a185cd..3f54cc5096 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -976,14 +976,6 @@ jobs:
uses: ./.github/actions/prepare_breeze_and_image
- name: "Helm Unit Tests: ${{ matrix.helm-test-package }}"
run: breeze testing helm-tests --helm-test-package "${{
matrix.helm-test-package }}"
- - name: >
- Post Tests success: Helm"
- uses: ./.github/actions/post_tests_success
- if: success()
- - name: >
- Post Tests failure: Helm"
- uses: ./.github/actions/post_tests_failure
- if: failure()
tests-postgres:
timeout-minutes: 130
