From bc0c1bc6f2c88cc7e8c97ea43b8e04a4d00d18df Mon Sep 17 00:00:00 2001 From: Jeff Ching Date: Thu, 26 Mar 2026 22:28:58 -0700 Subject: [PATCH] =?UTF-8?q?Revert=20"ci:=20only=20run=20default=20list=20o?= =?UTF-8?q?f=20graalvm=20tests=20if=20too=20many=20modules=20are=20to?= =?UTF-8?q?=E2=80=A6"?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This reverts commit d9e2e80d4bc8c1101034294a277011494dab2c39. --- .kokoro/common.sh | 16 ++++++++-------- .kokoro/presubmit/graalvm-native-a-presubmit.cfg | 5 +++++ .kokoro/presubmit/graalvm-native-b-presubmit.cfg | 5 +++++ .kokoro/presubmit/graalvm-native-c-presubmit.cfg | 5 +++++ 4 files changed, 23 insertions(+), 8 deletions(-) diff --git a/.kokoro/common.sh b/.kokoro/common.sh index 188dbfbb4443..3b3c8745febc 100644 --- a/.kokoro/common.sh +++ b/.kokoro/common.sh @@ -344,20 +344,20 @@ function run_graalvm_tests() { function generate_graalvm_presubmit_modules_list() { modules_assigned_list=() generate_modified_modules_list - if [[ ${#modified_module_list[@]} -gt 4 ]]; then - # Too many modules modified, run a subset - echo "Too many modules modified, running a subset" - module_list="java-aiplatform,java-compute" - elif [[ ${#modified_module_list[@]} -gt 0 ]]; then + if [[ ${#modified_module_list[@]} -gt 0 && ${#modified_module_list[@]} -lt 5 ]]; then # If only a few modules have been modified, focus presubmit testing only on them. module_list=$( IFS=, echo "${modified_module_list[*]}" ) else - # no modules modified - echo "No modules modified" - module_list="" + # If no modules have been modified or if too many have been modified, just test the modules + # specified in the MAVEN_MODULES env var. + if [ -z "${MAVEN_MODULES}" ]; then + echo "MAVEN_MODULES not defined in environment." + exit 1 + fi + module_list=${MAVEN_MODULES} fi } diff --git a/.kokoro/presubmit/graalvm-native-a-presubmit.cfg b/.kokoro/presubmit/graalvm-native-a-presubmit.cfg index 4311598f04ff..d61c074a7657 100644 --- a/.kokoro/presubmit/graalvm-native-a-presubmit.cfg +++ b/.kokoro/presubmit/graalvm-native-a-presubmit.cfg @@ -31,3 +31,8 @@ env_vars: { key: "SECRET_MANAGER_KEYS" value: "cloud-java-ci-it-service-account, java-bigqueryconnection-samples-secrets" } + +env_vars: { + key: "MAVEN_MODULES" + value: "java-aiplatform,java-compute" +} \ No newline at end of file diff --git a/.kokoro/presubmit/graalvm-native-b-presubmit.cfg b/.kokoro/presubmit/graalvm-native-b-presubmit.cfg index b0e48b147a09..d795cdb8fe8c 100644 --- a/.kokoro/presubmit/graalvm-native-b-presubmit.cfg +++ b/.kokoro/presubmit/graalvm-native-b-presubmit.cfg @@ -31,3 +31,8 @@ env_vars: { key: "SECRET_MANAGER_KEYS" value: "cloud-java-ci-it-service-account, java-bigqueryconnection-samples-secrets" } + +env_vars: { + key: "MAVEN_MODULES" + value: "java-aiplatform,java-compute" +} \ No newline at end of file diff --git a/.kokoro/presubmit/graalvm-native-c-presubmit.cfg b/.kokoro/presubmit/graalvm-native-c-presubmit.cfg index ba577630c304..b3a291e09966 100644 --- a/.kokoro/presubmit/graalvm-native-c-presubmit.cfg +++ b/.kokoro/presubmit/graalvm-native-c-presubmit.cfg @@ -31,3 +31,8 @@ env_vars: { key: "SECRET_MANAGER_KEYS" value: "cloud-java-ci-it-service-account, java-bigqueryconnection-samples-secrets" } + +env_vars: { + key: "MAVEN_MODULES" + value: "java-aiplatform,java-compute" +} \ No newline at end of file