RFR: 8342988: GHA: Build JTReg in single step [v2]

Magnus Ihse Bursie ihse at openjdk.org
Mon Oct 28 14:12:22 UTC 2024


On Fri, 25 Oct 2024 17:12:52 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

>> It is visible in current GHA runs that building jtreg sometimes break on dependencies checkout, for example:
>> 
>> 
>> [build.sh][INFO] CYGWIN_OR_MSYS=0
>> [build.sh][INFO] JAVA_HOME: /usr/lib/jvm/temurin-17-jdk-amd64
>> [build.sh][INFO] Downloading https://archive.apache.org/dist/ant/binaries/apache-ant-1.10.8-bin.zip to /home/runner/work/jdk/jdk/jtreg/src/make/../build/deps/apache-ant-1.10.8-bin.zip
>> Error: sh][ERROR] wget exited with exit code 4
>> Error: Process completed with exit code 1.
>> 
>> 
>> I suppose this is a side-effect of pulling lots of dependencies all at once. Building jtreg on every platform exposes us to these issues more often. Additionally, we had problems with building jtreg on various platforms. 
>> 
>> We do not have to build jtreg for every platform. We can just have a job step that does it once, and stores it in the cache for all dependent jobs to use. It would also save some build time across various jobs.
>
> Aleksey Shipilev has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Rework for artifacts

Ship it!

-------------

Marked as reviewed by ihse (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/21692#pullrequestreview-2399195911


More information about the build-dev mailing list