[airflow] branch master updated: Constraints generation runs regardless from test status (#11838)

[po...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

potiuk pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/master by this push:
     new 73884e8  Constraints generation runs regardless from test status (#11838)
73884e8 is described below

commit 73884e8e3b295e7e5ff40c5585886e29aac57fbe
Author: Jarek Potiuk <ja...@polidea.com>
AuthorDate: Tue Oct 27 10:19:30 2020 +0100

    Constraints generation runs regardless from test status (#11838)
    
    Constraints generation did not run when tests were failing
    but this was wrong. In case of tests that fail due to constraints
    upgrade we neeed to see the output of constraint generation, to
    know which constraints failed. Only pushing the constraint
    should be limited to the case where everything succeeded.
    
    This should make investigation of problems similar to #11837 much
    easier.
---
 .github/workflows/ci.yml | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 498e7fa..8f6a7da 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -893,18 +893,11 @@ jobs:
       fail-fast: false
     needs:
       - build-info
-      - static-checks
-      - static-checks-pylint
-      - tests-sqlite
-      - tests-mysql
-      - tests-postgres
-      - tests-kubernetes
     env:
       PYTHON_MAJOR_MINOR_VERSION: ${{ matrix.python-version }}
     if: >
       needs.build-info.outputs.basic-checks-only == 'false' &&
       (github.ref == 'refs/heads/master' || github.ref == 'refs/heads/v1-10-test' ) &&
-      github.event_name != 'pull' &&
       (github.repository == 'apache/airflow' || github.event_name != 'schedule')
     steps:
       - name: "Checkout ${{ github.ref }} ( ${{ github.sha }} )"
@@ -929,11 +922,18 @@ jobs:
     timeout-minutes: 10
     name: "Constraints push"
     runs-on: ubuntu-latest
-    needs: [build-info, constraints]
+    needs:
+      - build-info
+      - constraints
+      - static-checks
+      - static-checks-pylint
+      - tests-sqlite
+      - tests-mysql
+      - tests-postgres
+      - tests-kubernetes
     if: >
       needs.build-info.outputs.basic-checks-only == 'false' &&
       (github.ref == 'refs/heads/master' || github.ref == 'refs/heads/v1-10-test' ) &&
-      github.event_name != 'pull' &&
       (github.repository == 'apache/airflow' || github.event_name != 'schedule')
     steps:
       - name: "Checkout ${{ github.ref }} ( ${{ github.sha }} )"