bazelbuild / java_tools

Apache License 2.0
12 stars 8 forks source link

Release: java_tools v13.3 #86

Closed keertk closed 9 months ago

keertk commented 9 months ago

Link to relevant issue and/or commit:

  1. https://github.com/bazelbuild/rules_java/issues/156
  2. https://github.com/bazelbuild/bazel/pull/20294#issuecomment-1829923666

Other details: N/A

keertk commented 9 months ago

Step 2: https://buildkite.com/bazel-trusted/java-tools-binaries-java/builds/235

Step 3: COMMIT_HASH: 92d19ee3c346fd904bb5be68026c2be4914db340

...in progress (currently blocked by failures)

keertk commented 9 months ago

Update: This is on hold until the network issues are resolved. (cc @hvadehra)

fmeum commented 9 months ago

@keertk Could we make this rules_java 7.3.1? The turbine changes are bug fixes for regressions introduced by 7.3.0.

keertk commented 9 months ago

@fmeum Sure. Is the same true for these changes as well? Or do we need separate releases?

fmeum commented 9 months ago

@fmeum Sure. Is the same true for these changes as well? Or do we need separate releases?

Yes, those are exactly the turbine fixes I was referring to.

hvadehra commented 9 months ago

For some reason the newly added test (test_header_compiler_direct_supports_release) fails, but only for darwin arm64 (which we didn't test for while merging the change):

FAILED: test_header_compiler_direct_supports_release

ERROR: /private/var/tmp/_bazel_buildkite/febd3a7cb30f371618a049296da6e0a9/sandbox/darwin-sandbox/4063/execroot/_main/_tmp/e5441b83c301643fae8009ed9b94bd78/workspace/pkg/BUILD:2:13: Compiling Java headers pkg/libb-hjar.jar (1 source file) failed: (Exit 1): turbine_direct_graal failed: error executing Turbine command (from target //pkg:b) external/rules_java~7.3.0~toolchains~remote_java_tools_darwin_arm64/java_tools/turbine_direct_graal '-Dturbine.ctSymPath=external/rules_java~7.3.0~toolchains~remotejdk21_macos_aarch64/lib/ct.sym' ... (remaining 30 arguments skipped)

Use --sandbox_debug to see verbose messages from the sandbox and retain the sandbox build root for debugging
java.lang.NullPointerException: attempted to use --release, but JAVA_HOME is not set
    at java.base@20.0.2/java.util.Objects.requireNonNull(Objects.java:259)
    at com.google.turbine.binder.CtSymClassBinder.bind(CtSymClassBinder.java:52)
    at com.google.turbine.main.Main.bootclasspath(Main.java:309)
    at com.google.turbine.main.Main.compile(Main.java:142)
    at com.google.turbine.main.Main.compile(Main.java:133)
    at com.google.turbine.main.Main.main(Main.java:89)
Target //pkg:a failed to build

@fmeum any ideas?

fmeum commented 9 months ago

@hvadehra Yes, that's because there was no override for darwin_arm64 and I also didn't include it with my PR. This should be fixed by adding a corresponding line here and in the other tests with this setup: https://github.com/bazelbuild/bazel/blob/4a29f0851d1cde0240793cdc7a2e2cab926d31b7/src/test/shell/bazel/bazel_java_test.sh#L82

hvadehra commented 9 months ago

Ah, just a typo s/remote_java_tools_arm64/remote_java_tools_darwin_arm64/

keertk commented 9 months ago

Step 2: https://buildkite.com/bazel-trusted/java-tools-binaries-java/builds/242

Step 3: COMMIT_HASH: 4b13eee7b45a06fca1224ab5ce04d3e937ce35b0

Step 4:

{
  "version": "v13.3",
  "release": "false",
  "artifacts": {
    "java_tools_linux": {
      "mirror_url": "https://mirror.bazel.build/bazel_java_tools/release_candidates/java/v13.3/java_tools_linux-v13.3-rc1.zip",
      "sha": "a781eb28bb28d1fd9eee129272f7f2eaf93cd272f974a5b3f6385889538d3408"
    },
    "java_tools_windows": {
      "mirror_url": "https://mirror.bazel.build/bazel_java_tools/release_candidates/java/v13.3/java_tools_windows-v13.3-rc1.zip",
      "sha": "8fc29a5e34e91c74815c4089ed0f481a7d728a5e886c4e5e3b9bcd79711fee3d"
    },
    "java_tools_darwin_x86_64": {
      "mirror_url": "https://mirror.bazel.build/bazel_java_tools/release_candidates/java/v13.3/java_tools_darwin_x86_64-v13.3-rc1.zip",
      "sha": "55bd36bf2fad897d9107145f81e20a549a37e4d9d4c447b6915634984aa9f576"
    },
    "java_tools_darwin_arm64": {
      "mirror_url": "https://mirror.bazel.build/bazel_java_tools/release_candidates/java/v13.3/java_tools_darwin_arm64-v13.3-rc1.zip",
      "sha": "276bb552ee03341f93c0c218343295f60241fe1d32dccd97df89319c510c19a1"
    },
    "java_tools": {
      "mirror_url": "https://mirror.bazel.build/bazel_java_tools/release_candidates/java/v13.3/java_tools-v13.3-rc1.zip",
      "sha": "30a7d845bec3dd054ac45b5546c2fdf1922c0b1040b2a13b261fcc2e2d63a2f4"
    }
  }
}

Step 5: Commit: https://github.com/bazelbuild/rules_java/commit/2c45d348376c1e302fd6534de38a9b6c570493f2 2c45d348376c1e302fd6534de38a9b6c570493f2

Step 6 / 7: PR: https://github.com/bazelbuild/bazel/pull/20393 Presubmit: https://buildkite.com/bazel/bazel-bazel-github-presubmit/builds/18721

Step 8: Downstream build: https://buildkite.com/bazel/bazel-at-head-plus-downstream/builds/3463

Step 9:

{
  "version": "v13.3",
  "release": "true",
  "artifacts": {
    "java_tools_linux": {
      "mirror_url": "https://mirror.bazel.build/bazel_java_tools/releases/java/v13.3/java_tools_linux-v13.3.zip",
      "github_url": "https://github.com/bazelbuild/java_tools/releases/download/java_v13.3/java_tools_linux-v13.3.zip",
      "sha": "a781eb28bb28d1fd9eee129272f7f2eaf93cd272f974a5b3f6385889538d3408"
    },
    "java_tools_windows": {
      "mirror_url": "https://mirror.bazel.build/bazel_java_tools/releases/java/v13.3/java_tools_windows-v13.3.zip",
      "github_url": "https://github.com/bazelbuild/java_tools/releases/download/java_v13.3/java_tools_windows-v13.3.zip",
      "sha": "8fc29a5e34e91c74815c4089ed0f481a7d728a5e886c4e5e3b9bcd79711fee3d"
    },
    "java_tools_darwin_x86_64": {
      "mirror_url": "https://mirror.bazel.build/bazel_java_tools/releases/java/v13.3/java_tools_darwin_x86_64-v13.3.zip",
      "github_url": "https://github.com/bazelbuild/java_tools/releases/download/java_v13.3/java_tools_darwin_x86_64-v13.3.zip",
      "sha": "55bd36bf2fad897d9107145f81e20a549a37e4d9d4c447b6915634984aa9f576"
    },
    "java_tools_darwin_arm64": {
      "mirror_url": "https://mirror.bazel.build/bazel_java_tools/releases/java/v13.3/java_tools_darwin_arm64-v13.3.zip",
      "github_url": "https://github.com/bazelbuild/java_tools/releases/download/java_v13.3/java_tools_darwin_arm64-v13.3.zip",
      "sha": "276bb552ee03341f93c0c218343295f60241fe1d32dccd97df89319c510c19a1"
    },
    "java_tools": {
      "mirror_url": "https://mirror.bazel.build/bazel_java_tools/releases/java/v13.3/java_tools-v13.3.zip",
      "github_url": "https://github.com/bazelbuild/java_tools/releases/download/java_v13.3/java_tools-v13.3.zip",
      "sha": "30a7d845bec3dd054ac45b5546c2fdf1922c0b1040b2a13b261fcc2e2d63a2f4"
    }
  }
}

Step 10: java_tools released: https://github.com/bazelbuild/java_tools/releases/tag/java_v13.3

Step 11: rules_java PR: https://github.com/bazelbuild/rules_java/pull/157

Step 12: rules_java released: https://github.com/bazelbuild/rules_java/releases/tag/7.3.1 BCR PR: https://github.com/bazelbuild/bazel-central-registry/pull/1163

keertk commented 9 months ago

Done - https://github.com/bazelbuild/java_tools/releases/tag/java_v13.3 https://github.com/bazelbuild/rules_java/releases/tag/7.3.1