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