malthe commented on a change in pull request #18852:
URL: https://github.com/apache/airflow/pull/18852#discussion_r726074697
##########
File path: scripts/ci/testing/ci_run_airflow_testing.sh
##########
@@ -92,14 +93,28 @@ function run_all_test_types_in_parallel() {
echo ""
echo "${COLOR_YELLOW}Integration tests will be run separately at
the end after cleaning up docker${COLOR_RESET}"
echo ""
- # Remove Integration from list of tests to run in parallel
- test_types_to_run="${test_types_to_run//Integration/}"
- run_integration_tests_separately="true"
if [[ ${BACKEND} == "mssql" ]]; then
- # Skip running "Integration" tests for low memory condition for
mssql
- run_integration_tests_separately="false"
+ # Skip running "Integration" and "Providers" tests for low
memory condition for mssql
+ # Both might lead to memory issues even in run on their own.
We have no need to
+ # Test those specifically for MSSQL (and they will be tested
in `main` as there
+ # We have no memory limits
+ test_types_to_run="${test_types_to_run//Integration/}"
+ run_integration_tests_separately="false"
+ test_types_to_run="${test_types_to_run//Providers/}"
Review comment:
Merge error?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
