potiuk commented on code in PR #30705:
URL: https://github.com/apache/airflow/pull/30705#discussion_r1170578263
##########
dev/breeze/src/airflow_breeze/utils/selective_checks.py:
##########
@@ -606,7 +606,43 @@ def parallel_test_types(self) -> str:
)
test_types_to_remove.add(test_type)
current_test_types = current_test_types - test_types_to_remove
- return " ".join(sorted(current_test_types))
+ for test_type in tuple(current_test_types):
+ if test_type == "Providers":
+ current_test_types.remove(test_type)
+ current_test_types.update(
+ ("Providers[amazon]", "Providers[google]",
"Providers[-amazon,google]")
+ )
+ elif test_type.startswith("Providers[") and "amazon" in test_type
or "google" in test_type:
+ current_test_types.remove(test_type)
+ if "amazon" in test_type:
+ current_test_types.add("Providers[amazon]")
+ if "google" in test_type:
+ current_test_types.add("Providers[google]")
Review Comment:
cc: @vandonr-amz - I just run the `pytest -n auto tests` for the fun of it.
This has always ben an interesting experience (and pretty much always a
different one).
This time It failed pytest collection:
```
2 skipped, 2520 warnings, 407 errors in 313.56s (0:05:13`
```
Most of them:
```
E botocore.exceptions.ClientError: An error occurred
(UnrecognizedClientException) when calling the GetParameter operation: The
security token included in the request is invalid.
```
Seems that AWS tests have problems to be even collected in parallel :)
--
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: commits-unsubscr...@airflow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
