Skip to content

Commit d9e2e80

Browse files
authored
ci: only run default list of graalvm tests if too many modules are touched (googleapis#12216)
Fixes googleapis#12215
1 parent 435dd8c commit d9e2e80

File tree

4 files changed

+8
-23
lines changed

4 files changed

+8
-23
lines changed

.kokoro/common.sh

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -344,20 +344,20 @@ function run_graalvm_tests() {
344344
function generate_graalvm_presubmit_modules_list() {
345345
modules_assigned_list=()
346346
generate_modified_modules_list
347-
if [[ ${#modified_module_list[@]} -gt 0 && ${#modified_module_list[@]} -lt 5 ]]; then
347+
if [[ ${#modified_module_list[@]} -gt 4 ]]; then
348+
# Too many modules modified, run a subset
349+
echo "Too many modules modified, running a subset"
350+
module_list="java-aiplatform,java-compute"
351+
elif [[ ${#modified_module_list[@]} -gt 0 ]]; then
348352
# If only a few modules have been modified, focus presubmit testing only on them.
349353
module_list=$(
350354
IFS=,
351355
echo "${modified_module_list[*]}"
352356
)
353357
else
354-
# If no modules have been modified or if too many have been modified, just test the modules
355-
# specified in the MAVEN_MODULES env var.
356-
if [ -z "${MAVEN_MODULES}" ]; then
357-
echo "MAVEN_MODULES not defined in environment."
358-
exit 1
359-
fi
360-
module_list=${MAVEN_MODULES}
358+
# no modules modified
359+
echo "No modules modified"
360+
module_list=""
361361
fi
362362
}
363363

.kokoro/presubmit/graalvm-native-a-presubmit.cfg

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -31,8 +31,3 @@ env_vars: {
3131
key: "SECRET_MANAGER_KEYS"
3232
value: "cloud-java-ci-it-service-account, java-bigqueryconnection-samples-secrets"
3333
}
34-
35-
env_vars: {
36-
key: "MAVEN_MODULES"
37-
value: "java-aiplatform,java-compute"
38-
}

.kokoro/presubmit/graalvm-native-b-presubmit.cfg

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -31,8 +31,3 @@ env_vars: {
3131
key: "SECRET_MANAGER_KEYS"
3232
value: "cloud-java-ci-it-service-account, java-bigqueryconnection-samples-secrets"
3333
}
34-
35-
env_vars: {
36-
key: "MAVEN_MODULES"
37-
value: "java-aiplatform,java-compute"
38-
}

.kokoro/presubmit/graalvm-native-c-presubmit.cfg

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -31,8 +31,3 @@ env_vars: {
3131
key: "SECRET_MANAGER_KEYS"
3232
value: "cloud-java-ci-it-service-account, java-bigqueryconnection-samples-secrets"
3333
}
34-
35-
env_vars: {
36-
key: "MAVEN_MODULES"
37-
value: "java-aiplatform,java-compute"
38-
}

0 commit comments

Comments
 (0)