Automated rollback of commit 01f620599c6138de4b4551da92917148ab18efe3. *** Reason for rollback *** Fix jdeps_test RBE failure The culprit change added a new class that presumably pushed us over the xargs arg limit. Unfortunately, jdeps does not support @argfile, so this change updates the command buffer size to twice the default on RBE. Fixes https://github.com/bazelbuild/bazel/issues/20288 *** Original change description *** Disable //src/test/shell/bazel:jdeps_test on RBE build to fix presubmit Related https://github.com/bazelbuild/bazel/issues/20288 PiperOrigin-RevId: 584820138 Change-Id: I6f5157d7e72c539a5fdc81841a35543df473aa4a
diff --git a/.bazelci/presubmit.yml b/.bazelci/presubmit.yml index 2b61058..e10485c 100644 --- a/.bazelci/presubmit.yml +++ b/.bazelci/presubmit.yml
@@ -475,8 +475,6 @@ - "-//src/test/shell/bazel/android:aapt_integration_test" - "-//src/test/shell/bazel/android:aapt_integration_test_with_head_android_tools" - "-//src/test/shell/bazel/android:aapt_integration_test_with_platforms" - # https://github.com/bazelbuild/bazel/issues/20288 - - "-//src/test/shell/bazel:jdeps_test" include_json_profile: - build - test
diff --git a/src/test/shell/bazel/jdeps_test.sh b/src/test/shell/bazel/jdeps_test.sh index 9851340..2776ebe 100755 --- a/src/test/shell/bazel/jdeps_test.sh +++ b/src/test/shell/bazel/jdeps_test.sh
@@ -72,7 +72,7 @@ # src/test/shell/bazel/jdeps_class_denylist.txt. find . -type f -iname \*class | \ grep -vFf "$denylist" | \ - xargs ../jdk/reduced/bin/jdeps --list-reduced-deps --ignore-missing-deps | \ + xargs -s 262144 ../jdk/reduced/bin/jdeps --list-reduced-deps --ignore-missing-deps | \ grep -v "unnamed module" > ../jdeps \ || fail "Failed to run jdeps on non denylisted class files." cd ..