RFR: 8355452: GHA: Test jtreg tier1 on linux-x64 static-jdk
Jiangli Zhou
jiangli at openjdk.org
Thu May 1 22:49:56 UTC 2025
Please review this PR that adds a `test-linux-x64-static` job, which runs tier1 tests on the static-jdk 'release' binary created from the `linux-x64-static` build job in GHA. Following are the details on the changes:
.github/actions/get-bundles/action.yml.
- Add `static-suffix` parameter. `static-suffix` is added to the bundle name.
- Add `static-jdk-path` output.
- Unpack static bundles in bundles/static-jdk.
.github/actions/upload-bundles/action.yml
- Add `static-suffix` parameter. The `static-suffix` is added to the bundle name. The `linux-x64-static` build job sets the parameter as "-static". In other jobs, `static-suffix` not set.
.github/workflows/build-linux.yml
- Pass `${{ inputs.static-suffix }}` to upload-bundles action, with the `static-suffix` parameter.
.github/workflows/main.yml
- Build `product-bundles test-bundles static-jdk-bundles` for `linux-x64-static` job.
- Add `test-linux-x64-static` job. It currently tests on `release` binary and not `debug` binary, since there are build issue with `debug` due to GHA resource/space limit.
- Set `debug-suffix` for the existing non-static test jobs, which test on `debug` binaries.
.github/workflows/test.yml
- Add `debug-suffix` parameter and replace `debug-suffix: -debug` with `debug-suffix: ${{ inputs.debug-suffix }}` in hs/tier1 tests in the test matrix. The existing test jobs (on non-static JDK) set `debug-suffix` to `-debug` to test on `debug` binaries.
- Add `static-suffix` parameter. Add `${{ inputs.static-suffix }}` to the test result artifact name.
- Add `run-tests-static`.
- Add step for notifying test failures on static JDK.
@shipilev Could you please help review this change? Thanks!
-------------
Commit messages:
- Fix typo.
- Export static-problemlist-path in 'path' step.
- Remove inputs.static-suffix check when notify test failure.
- Replace $(pwd) with `pwd` to form the path to static JDK problemlist.
- Use different id fot test on static. Add step to notify failures on static JDK.
- - Change run-tests-on-static id to run-tests id.
- Don't set debug-suffix for static test job, since that causes incorrect bundle name used.
- - Switch back to build 'release' binary for linux-x64-static job.
- Switch to do 'debug' build for linux-x64-static job.
- - Only setup static_jdk_dir if inputs.static-suffix is not empty.
- ... and 14 more: https://git.openjdk.org/jdk/compare/62d165d0...5306dac3
Changes: https://git.openjdk.org/jdk/pull/24992/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=24992&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8355452
Stats: 108 lines in 5 files changed: 92 ins; 2 del; 14 mod
Patch: https://git.openjdk.org/jdk/pull/24992.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/24992/head:pull/24992
PR: https://git.openjdk.org/jdk/pull/24992
More information about the build-dev
mailing list