-
-
Notifications
You must be signed in to change notification settings - Fork 229
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bump preview (EA) JDK21 version to 21.0.4+5 #772
Bump preview (EA) JDK21 version to 21.0.4+5 #772
Conversation
I think that the EA versions need to be replaced by released versions in this repository as was done for the controllers in: Unfortunately, I'm not available at the moment to make that change. If others are interested in making the change, I'm happy to review the change. |
The JDK21 preview covers 2 Linux platforms: Line 320 in 8d8c683
For Same for the Docker image (which we build on top of): https://hub.docker.com/layers/library/eclipse-temurin/21.0.2_13-jdk-jammy/images/sha256-952185109e6790588902d524bd43fcf5f478e022847097ef9ae8ef5b82511bbd?context=explore => it means we can already move s390x out ofr "preview" for JDK21, but we'll have to decide (SIG Platform?) to keep or to drop support for arm32. |
Thanks for doing that investigation. The discussions in Platform SIG accepted that because Eclipse Temurin will not support armv7 after Java 17, the Jenkins project should not include armv7 images. Pull request says:
|
I'll work on the move to "standard" JDK21 builds for |
a25c8f7
to
b2ddb3b
Compare
5d9d2d4
to
9fdef66
Compare
9fdef66
to
ad5f7bd
Compare
ad5f7bd
to
3510800
Compare
f7f5925
to
1dd7376
Compare
76e65d5
to
3183ac2
Compare
3183ac2
to
5d4ce5c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The CI check fails with:
ERROR: failed to solve: process "/bin/bash -o pipefail -c set -x; apt-get update && apt-get install --no-install-recommends -y ca-certificates jq wget && DASHED_JAVA_VERSION=$(echo \"$JAVA_VERSION\" | sed -e 's/\\./-/g' -e 's/+/-/' -e 's/21-35/21-0-35/') && JAVA_MAJOR_VERSION=$(echo \"$JAVA_VERSION\" | cut -d'+' -f1 | cut -d'.' -f1) && ENCODED_JAVA_VERSION=$(echo \"$JAVA_VERSION\" | jq \"@uri\" -jRr) && CONVERTED_ARCH=$(arch | sed -e 's/x86_64/x64/' -e 's/armv7l/arm/') && wget --quiet [https://github.com/adoptium/temurin\"${JAVA_MAJOR_VERSION}\"-binaries/releases/download/jdk-\"${ENCODED_JAVA_VERSION}\"-ea-beta/OpenJDK\"${JAVA_MAJOR_VERSION}\"U-jdk_\"${CONVERTED_ARCH}\"_linux_hotspot_ea_\"${DASHED_JAVA_VERSION}\".tar.gz](https://github.com/adoptium/temurin/%22$%7BJAVA_MAJOR_VERSION%7D/%22-binaries/releases/download/jdk-/%22$%7BENCODED_JAVA_VERSION%7D/%22-ea-beta/OpenJDK/%22$%7BJAVA_MAJOR_VERSION%7D/%22U-jdk_/%22$%7BCONVERTED_ARCH%7D/%22_linux_hotspot_ea_/%22$%7BDASHED_JAVA_VERSION%7D/%22.tar.gz) -O /tmp/jdk.tar.gz && tar -xzf /tmp/jdk.tar.gz -C /opt/ && rm -f /tmp/jdk.tar.gz" did not complete successfully: exit code: 8
Failing for
|
5d4ce5c
to
a26a375
Compare
0d0051e
to
692ccbf
Compare
Still arm32 of course...
|
692ccbf
to
401f003
Compare
Still failing for
|
@gounthar what do you think if we open PR removing the |
... -bake.hcl file Made with ❤️️ by updatecli
401f003
to
e2b4ad6
Compare
Bump JDK21 preview version (EA) for all Linux images
Bump preview (EA) JDK21 version for Linux images in the docker-bake.hcl file
changes detected: path "variable.JAVA21_PREVIEW_VERSION.default" updated from "21.0.1+12" to "21.0.4+5" in file "docker-bake.hcl"
Created automatically by Updatecli
Options:
Most of Updatecli configuration is done via its manifest(s).
Feel free to report any issues at github.com/updatecli/updatecli.
If you find this tool useful, do not hesitate to star our GitHub repository as a sign of appreciation, and/or to tell us directly on our chat!