-
-
Notifications
You must be signed in to change notification settings - Fork 522
feat(Linux): switch supported architectures to GA Temurin jdk21 images #436
feat(Linux): switch supported architectures to GA Temurin jdk21 images #436
Conversation
04fde2c
to
9bc125c
Compare
Made with ❤️️ by updatecli, rebased by lemeurherve
…ew target in docker-bake so this target is picked up by the Makefile
…21_preview target in docker-bake so this target is picked up by the Makefile" This reverts commit 055ae83.
I had to revert 055ae83 as the jdk21-preview parent image doesn't include these architectures: https://hub.docker.com/layers/jenkins/agent/3180.v3dd999d24861-2-jdk21-preview/images/sha256-bede7232308fb7fe5432bc5cf72a58623676b3e01471f5ff965ef1dd3e8109ff?context=explore, which has for immediate result the fact the debian_jdk_21_image target isn't built anymore in this PR. To build this debian_jdk_21_preview image in builds, more modifications to the Makefile are required to include architectures other than linux/amd64 & linux/arm64. Something like we've already discussed with @dduportal, building each Linux variant on its own agent with corresponding architecture. (somehow like what's done for Windows images) |
Another easier (?) solution while we're keeping preview images with EA Temuring binaries would be to trigger another docker-agent release with its debian_jdk21_preview target also containing linux/md64 & linux/arm64 architectures. |
…es (jenkinsci#437) * fix(updatecli): check preview parent images only for some architectures * Update docker-agent.yaml
…k21-preview Dockerfiles (jenkinsci#439)
Updatecli is failing because of 54b2e12, will be restored when merged. |
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.
LGTM!
I'm not sure to understand what is the need to change Makefile? As you underlined, the problem come from the parent repository not publishing the |
In the Makefile the list of images to build is retrieved by https://github.com/jenkinsci/docker-inbound-agent/blob/master/Makefile#L47-L48 This rule excludes the preview image. The easiest solution to build it is to reintroduce amd64/arm64 architectures in debian_jdk_21_preview image of docker-agent, the alternative would be to change this listing (maybe?) |
As discussed directly:
|
…meurherve/feat-switch-to-official-ga-jdk21-temurin-images feat(Linux): switch supported architectures to GA Temurin jdk21 images
…meurherve/feat-switch-to-official-ga-jdk21-temurin-images feat(Linux): switch supported architectures to GA Temurin jdk21 images
…meurherve/feat-switch-to-official-ga-jdk21-temurin-images feat(Linux): switch supported architectures to GA Temurin jdk21 images
…meurherve/feat-switch-to-official-ga-jdk21-temurin-images feat(Linux): switch supported architectures to GA Temurin jdk21 images
…meurherve/feat-switch-to-official-ga-jdk21-temurin-images feat(Linux): switch supported architectures to GA Temurin jdk21 images
…meurherve/feat-switch-to-official-ga-jdk21-temurin-images feat(Linux): switch supported architectures to GA Temurin jdk21 images
…e/feat-switch-to-official-ga-jdk21-temurin-images feat(Linux): switch supported architectures to GA Temurin jdk21 images
Follow-up of jenkinsci/docker-agent#521
Needs #437 and the subsequent updatecli PR updating parent image version in Dockerfiles.
EDIT: cherry-picked corresponding changes.
Testing done
https://ci.jenkins.io/job/Packaging/job/docker-inbound-agent/job/PR-436/13/
Submitter checklist