potiuk commented on a change in pull request #12114:
URL: https://github.com/apache/airflow/pull/12114#discussion_r518289959
##########
File path: scripts/ci/selective_ci_checks.sh
##########
@@ -475,8 +475,11 @@ function get_count_kubernetes_files() {
echo
local pattern_array=(
"^airflow/kubernetes"
+ "^airflow/executors/kubernetes_executor.py"
"^chart"
- "^tests/kubernetes_tests"
+ "^tests/kubernetes"
Review comment:
This should not be there. Those "kubernetes_tests" are special tests
that are not the same as all the "unit tests". They do not need CI image,
instead they are using local virtual env to run tests and Kind-Kubernetes
started on the host using the PROD image.
```suggestion
```
##########
File path: scripts/ci/selective_ci_checks.sh
##########
@@ -475,8 +475,11 @@ function get_count_kubernetes_files() {
echo
local pattern_array=(
"^airflow/kubernetes"
+ "^airflow/executors/kubernetes_executor.py"
"^chart"
- "^tests/kubernetes_tests"
+ "^tests/kubernetes"
+ "^kubernetes_tests"
+ "^tests/executors/test_kubernetes_executor.py"
Review comment:
```suggestion
```
Same here.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
