On Fri, 25 Oct 2024 17:12:52 GMT, Aleksey Shipilev <sh...@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

Reply via email to