Bump chainguard static base image to latest sha#9490
Bump chainguard static base image to latest sha#9490ltwongaa wants to merge 4 commits intotektoncd:mainfrom
Conversation
Latest image in chainguard static image repo https://images.chainguard.dev/directory/image/static/versions
|
/kind bug |
|
/retest |
|
confirmed the digest resolves to a current image: |
|
/retest |
1 similar comment
|
/retest |
Seems E2E failed expecting an arm image, I changed to the image sha to the arm image one |
|
/retest |
|
/retest |
| @@ -1,4 +1,4 @@ | |||
| defaultBaseImage: cgr.dev/chainguard/static@sha256:67a1b00e0134e2b3a614c7198a26f7deed9d11b7acad4d52c79c0cfd47a2eae7 | |||
| defaultBaseImage: cgr.dev/chainguard/static@sha256:d6a97eb401cbc7c6d48be76ad81d7899b94303580859d396b52b67bc84ea7345 | |||
There was a problem hiding this comment.
If this sha corresponds to a tag, could you mention the tag in a comment please?
There was a problem hiding this comment.
Hi @afrittoli ,
chainguard now only provides the tag latest there,
all the others are with the sha.
This is the current sha of the latest tag.
FYI, if you view from the official repository, they hide all the other images except the latest.
https://images.chainguard.dev/directory/image/static/versions#/
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: afrittoli The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
vdemeester
left a comment
There was a problem hiding this comment.
The new digest is a valid manifest list (OCI image index), but it significantly reduces architecture coverage compared to the old one.
Old digest (67a1b00e...) — 7 architectures:
| Arch | OS |
|---|---|
| 386 | linux |
| amd64 | linux |
| arm/v6 | linux |
| arm/v7 | linux |
| arm64 | linux |
| ppc64le | linux |
| s390x | linux |
New digest (d6a97eb4...) — 2 architectures:
| Arch | OS |
|---|---|
| amd64 | linux |
| arm64 | linux |
This drops 386, arm/v6, arm/v7, ppc64le, and s390x. We do ship ppc64le and s390x so this would be a problem.
/hold
|
For context, Chainguard announced this change in May 2024 and applied it on July 15, 2024. They moved the free/Developer tier From their blog post:
The old digest ( Since we need those architectures, options include:
|
|
emmm, I wonder who would able to make the decision on the approaches |
I agree with you on the image not being up-to-date, we need to take a decision rather soon than later. |
|
Agree with the /hold. #9557 is the right path forward here. |
|
@ltwongaa can you please rebase this PR ? Thank you. |
|
I think we can close this as we are working towards owning the base image, see tektoncd/plumbing#3221 and #9576. Thank you for the contribution @ltwongaa 🤗 |
Manually bump ko build base image to latest image in chainguard static image repo
https://images.chainguard.dev/directory/image/static/versions
This fixes the os image out of support issue
#9163
Changes
Submitter Checklist
As the author of this PR, please check off the items in this checklist:
/kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tepRelease Notes