Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

A bunch of environment changes. #1745

Closed
wants to merge 0 commits into from
Closed

Conversation

copybara-service[bot]
Copy link

@copybara-service copybara-service bot commented May 17, 2023

A bunch of environment changes.

  • Bump minimum JRE to 11 (from 8).
  • Change testing matrix from {8,11,15,17} to {11,17,21-ea}. (Except the bazel build has trouble parsing "21-ea" as a java version, so s/21-ea/20 for bazel.)
  • Also test on macos (but only with jdk17). (Attempted to test on windows too, but there's issues right now.)
  • Cleanup the POMs a little bit (remove unused mailing list reference, fix CI link)
  • Bump bazel version to 6.2.0, otherwise the tests fail b/c they try to set a security manager (see Bazel@HEAD on JDK 17: GoogleTestSecurityManager WARNING: System::setSecurityManager will be removed in a future release bazelbuild/bazel#14502, which was fixed in bazelbuild/bazel@7556e11, which was cherrypicked into the 6.2.0 release). This also required suppressing the "BanJNDI" errorprone check in the JNDI extension, which was also added to 6.2.0. (A future change should probably just remove the JNDI extension altogether.)

@copybara-service copybara-service bot force-pushed the test_532797192 branch 4 times, most recently from 0e3d89d to 533cfd2 Compare May 17, 2023 18:10
@copybara-service copybara-service bot closed this May 17, 2023
@copybara-service copybara-service bot deleted the test_532797192 branch May 17, 2023 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

0 participants