-
-
Notifications
You must be signed in to change notification settings - Fork 232
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
JDK 21 preview updatecli manifest is failing #798
Comments
|
The error is clearly related to the shell script:
=> worth executing it in debug over there |
The script is failing, but the URL is valid. 🤔 |
/~https://github.com/adoptium/temurin21-binaries/releases/download/jdk-21.0.4%2B1-ea-beta/OpenJDK21U-jdk_x64_linux_hotspot_21.0.4_1-ea.tar.gz was returning a 404 error yesterday 🤔 Maybe a GitHub intermittent issue that would explain why it succeeded sometimes? |
FTR, the URL from #772 isn't the same cf https://ci.jenkins.io/job/Packaging/job/docker-agent/job/PR-772/16/pipeline-console/?selected-node=102 - /~https://github.com/adoptium/temurin21-binaries/releases/download/jdk-21.0.4%2B1-ea-beta/OpenJDK21U-jdk_x64_linux_hotspot_21.0.4_1-ea.tar.gz
+ /~https://github.com/adoptium/temurin21-binaries/releases/download/jdk-21.0.4%2B1-ea-beta/OpenJDK21U-jdk_arm_linux_hotspot_ea_21-0-4-1.tar.gz |
Is the error reproducible locally? |
No (cf #798 (comment)) Not reproduced in CI anymore and version correctly bumped by updatecli in #772. Important note, this jdk21 preview manifest isn't using the adoptium scripts to retrieve the JDK download link (it was introduced before them): /~https://github.com/jenkinsci/docker-agent/blob/master/updatecli/updatecli.d/jdk21-preview.yaml The adoptium scripts need additional logic if we want to use them for that as this preview version is an "EA" one, not a "GA". I'd suggest fixing this jdk21 preview manifest to return the correct URL for now instead of increasing the complexity of the adoptium scripts immediately. |
Nor the jdk21 preview Dockerfile cf docker-agent/debian/preview/Dockerfile Lines 15 to 27 in d473399
Last URL from https://ci.jenkins.io/job/Packaging/job/docker-agent/job/PR-772/25/pipeline-console/?selected-node=94: /~https://github.com/adoptium/temurin21-binaries/releases/download/jdk-21.0.4%2B2-ea-beta/OpenJDK21U-jdk_arm_linux_hotspot_ea_21-0-4-2.tar.gz (404 not found)
or the jdk21 Dockerfile logic. |
21.0.4+2 is absent from /~https://github.com/adoptium/temurin21-binaries/releases 🤔 |
I don't understand what the problem is then. The updatecli manifest is made to retrieve the "latest" version matching a certain pattern as source, and the condition verify that this version has the proper binaries available (because between creating a tag and having all binary released, a lot of time can pass). The partial log output you've added in the body message shows that the condition failed, which is usually because the binaries are NOT available (yet).
Yes, that is and always has been the whole puprose of running the check in an "update condition". => What is the problem we want to solve, considering all of this thread? I'm asking because it looks like the error was "transient" (which happens on each Temurin release as they don't release everything immediately) so the issue might be closeable but I might be missing something? |
This issue can be closed but the jdk21 Dockerfile needs a fix as the final URL called in the image build /~https://github.com/adoptium/temurin21-binaries/releases/download/jdk-21.0.4%2B2-ea-beta/OpenJDK21U-jdk_arm_linux_hotspot_ea_21-0-4-2.tar.gz is invalid. |
Jenkins and plugins versions report
N/A
What Operating System are you using (both controller, and any agents involved in the problem)?
N/A
Reproduction steps
Observe updatecli actions logs in this repository.
Expected Results
No error in logs.
Actual Results
Error in logs:
Anything else?
No response
Are you interested in contributing a fix?
No response
The text was updated successfully, but these errors were encountered: