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 6e80bcd463 Only run mssql tests on self-hosted runners (#30831)
6e80bcd463 is described below

commit 6e80bcd463a701b30ce42f4499ea45d4c4e47972
Author: Jarek Potiuk <[email protected]>
AuthorDate: Mon Apr 24 09:46:25 2023 +0200

    Only run mssql tests on self-hosted runners (#30831)
    
    The MSSQL tests are rather flaky on public runners. They also
    almost never bring any extra value when fail usually those are
    flaky behaviour of MSSQL database for those runners due to
    resource constraints.
    
    This PR changes MSSQL tests to be only run for self-hosted runners,
    which means that they will be run for PRs of committers and in
    canary builds.
    
    This should be enough to find out potentially mssql-only breaking
    changes and fix them - without disturbing regular contributors.
---
 .github/workflows/ci.yml | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index ef12395c17..1e4d476ff2 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -1127,7 +1127,7 @@ jobs:
       BACKEND_VERSION: "${{matrix.mssql-version}}"
       JOB_ID: "mssql-${{matrix.mssql-version}}-${{matrix.python-version}}"
       COVERAGE: "${{needs.build-info.outputs.run-coverage}}"
-    if: needs.build-info.outputs.run-tests == 'true'
+    if: needs.build-info.outputs.run-tests == 'true' && 
needs.build-info.outputs.runs-on == 'self-hosted'
     steps:
       - name: Cleanup repo
         shell: bash
@@ -1334,7 +1334,6 @@ jobs:
       - tests-postgres
       - tests-sqlite
       - tests-mysql
-      - tests-mssql
       - tests-quarantined
       - tests-integration-postgres
       - tests-integration-mysql
@@ -1499,7 +1498,6 @@ jobs:
       - static-checks
       - tests-sqlite
       - tests-mysql
-      - tests-mssql
       - tests-postgres
       - tests-integration-postgres
       - tests-integration-mysql
@@ -1662,7 +1660,6 @@ jobs:
       - static-checks
       - tests-sqlite
       - tests-mysql
-      - tests-mssql
       - tests-postgres
       - tests-integration-postgres
       - tests-integration-mysql

Reply via email to