android_local_test isn't compatible with Android Platforms

[katre]

Forked from #18262.

The android_local_test rule is a strange beast which wants to do two things: it wants to build an APK, using a valid Android SDK.

But, it also wants to run a Java binary on the local platform, with that APK as a data file. This worked before #18262, but now that we are being honest about Java versions, the cracks show up.

This sounds like a case for execution groups to me, but I also don't actually understand how this works under the hood.

For now, this means that we need to disable testing android_local_test with Android Platforms, and get the code cleaned up.

We probably also need to fix the Starlark version in bazelbuild/rules_android.

[gregestren]

Is this a blocker for flipping the flag? If one of the android rules would fail with it?

[katre]

I've marked it as a blocker for Bazel 7 release (not for starting the release, but for finishing it), but we need someone from the rules_android team to investigate because the code is pretty confusing.

[iancha1992]

@bazel-io fork 7.0.0

[katre]

Error log from bazel test //src/test/shell/bazel/android:android_local_test_integration_test_with_platforms (with the test re-enabled):

** test_android_local_test *****************************************************
$TEST_TMPDIR defined: output root default is '/usr/local/google/home/jcater/.cache/bazel/_bazel_jcater/a85a0dacd9e20e18c47928db0f1530e0/sandbox/linux-sandbox/1280/execroot/_main/_tmp/c0008185402992cea4ecb54c46211ee6' and max_idle_secs default is '15'.
WARNING: The following rc files are no longer being read, please transfer their contents or import their path into one of the standard rc files:
/etc/bazel.bazelrc
Extracting Bazel installation...
Starting local Bazel server and connecting to it...
INFO: Starting clean (this may take a while). Consider using --async if the clean takes more than several minutes.
-- Test log: -----------------------------------------------------------
$TEST_TMPDIR defined: output root default is '/usr/local/google/home/jcater/.cache/bazel/_bazel_jcater/a85a0dacd9e20e18c47928db0f1530e0/sandbox/linux-sandbox/1280/execroot/_main/_tmp/c0008185402992cea4ecb54c46211ee6' and max_idle_secs default is '15'.
WARNING: The following rc files are no longer being read, please transfer their contents or import their path into one of the standard rc files:
/etc/bazel.bazelrc
Computing main repo mapping: 
WARNING: --enable_bzlmod is set, but no MODULE.bazel file was found at the workspace root. Bazel will create an empty MODULE.bazel file. Please consider migrating your external dependencies from WORKSPACE to MODULE.bazel. For more details, please refer to https://github.com/bazelbuild/bazel/issues/18958.
Loading: 
Loading: 0 packages loaded
Analyzing: target //javatests/com/bin:test (1 packages loaded, 0 targets configured)
Analyzing: target //javatests/com/bin:test (1 packages loaded, 0 targets configured)
[0 / 1] checking cached actions
Analyzing: target //javatests/com/bin:test (5 packages loaded, 6 targets configured)
[1 / 1] checking cached actions
Analyzing: target //javatests/com/bin:test (62 packages loaded, 12 targets configured)
[1 / 1] checking cached actions
ERROR: /usr/local/google/home/jcater/.cache/bazel/_bazel_jcater/a85a0dacd9e20e18c47928db0f1530e0/sandbox/linux-sandbox/1280/execroot/_main/_tmp/c0008185402992cea4ecb54c46211ee6/workspace.FJHlJe57/javatests/com/bin/BUILD:5:19: While resolving toolchains for target //javatests/com/bin:test (d3e1c6f): No matching toolchains found for types @bazel_tools//tools/jdk:runtime_toolchain_type.
To debug, rerun with --toolchain_resolution_debug='@bazel_tools//tools/jdk:runtime_toolchain_type'
If platforms or toolchains are a new concept for you, we'd encourage reading https://bazel.build/concepts/platforms-intro.
ERROR: Analysis of target '//javatests/com/bin:test' failed; build aborted
INFO: Elapsed time: 3.703s, Critical Path: 0.04s
INFO: 1 process: 1 internal.
ERROR: Build did NOT complete successfully
ERROR: No test targets were found, yet testing was requested
------------------------------------------------------------------------
test_android_local_test FAILED: Tests for //javatests/com/bin:test failed.
/usr/local/google/home/jcater/.cache/bazel/_bazel_jcater/a85a0dacd9e20e18c47928db0f1530e0/sandbox/linux-sandbox/1280/execroot/_main/bazel-out/k8-fastbuild/bin/src/test/shell/bazel/android/android_local_test_integration_test_with_platforms.runfiles/_main/src/test/shell/bazel/android/android_local_test_integration_test_with_platforms:162: in call to test_android_local_test

Tear down:

INFO[android_local_test_integration_test_with_platforms 2023-11-06 21:16:31 (+0000)] Cleaning up workspace

FAILED: test_android_local_test

The problem (I think) is that the test sets --platforms to an Android platform (in android_helper.sh), but there's no JDK available for that platform.

[Wyverald]

Hullo! We're trying to get 7.0 blockers under control and this one seems to not have been investigated yet. Could we get an initial triage and an estimate on how long this'll take to fix?

[katre]

@ahumesky was investigating, I'll ask him to update.

[ahumesky]

I have a change out for review that should address this