RFR: 8355452: GHA: Test jtreg tier1 on linux-x64 static-jdk [v5]
Jiangli Zhou
jiangli at openjdk.org
Tue May 13 01:18:58 UTC 2025
On Mon, 5 May 2025 20:59:08 GMT, Jiangli Zhou <jiangli at openjdk.org> wrote:
>> .github/workflows/main.yml line 234:
>>
>>> 232: with:
>>> 233: platform: linux-x64
>>> 234: make-target: 'product-bundles test-bundles static-jdk-bundles'
>>
>> This will make us build the tests and the normal JDK twice, once here and once for the normal build-linux-x64 target. That seems like a useless waste of time and computing resources. There must be a better way of doing this.
>
> I haven't tried, but I think it's possible to not requiring building `product-bundles`. The `product-bundles` (regular JDK) is used for compiling the test java sources. I think it's possible to use the boot JDK. I'll try that.
I removed `product-bundles test-bundles` as suggested. Thanks.
With your other suggestion to add `build-linux-x64` to the `needs` list for `test-linux-x64-static`, I didn't have to change to use the boot JDK for compiling the test java sources, since we download the bundles from the `build-linux-x64` as well and have a regular JDK. `JDK_FOR_COMPILE` is still set to `${{ steps.bundles.outputs.jdk-path }}`. That's actually better than using the boot JDK for `JDK_FOR_COMPILE`, since tests may use tools from the `-Dcompile.jdk` specified path. The boot JDK is not built from the current source, and using tools from the boot JDK may cause unexpected results.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24992#discussion_r2085755003
More information about the build-dev
mailing list