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

Aleksey Shipilev shade at openjdk.org
Fri Oct 25 17:12:52 UTC 2024


> 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

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/21692/files
  - new: https://git.openjdk.org/jdk/pull/21692/files/21f65c3f..45288ca4

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=21692&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=21692&range=00-01

  Stats: 110 lines in 7 files changed: 69 ins; 35 del; 6 mod
  Patch: https://git.openjdk.org/jdk/pull/21692.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/21692/head:pull/21692

PR: https://git.openjdk.org/jdk/pull/21692


More information about the build-dev mailing list