Skip to content
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

Update Distroless to Debian 12 #3432

Merged
merged 1 commit into from
Mar 28, 2024
Merged

Conversation

haydentherapper
Copy link
Contributor

@haydentherapper haydentherapper commented Mar 25, 2024

Overview

Updated the Distroless image to Debian 12

What this PR does / why we need it

gcr.io/distroless/static is Debian 10, which is quite outdated. Updated to the Debian 12 version of the static image, and used the nonroot tag since the root user isn't needed.

We've been running a version of Dex in production with this image without issue.

Special notes for your reviewer

gcr.io/distroless/static is Debian 10, which is quite outdated. Updated to the Debian 12 version of the static image, and used the nonroot tag since the root user isn't needed.

We've been running a version of Dex in production with this image without issue.

Signed-off-by: Hayden B <hblauzvern@google.com>
@nabokihms
Copy link
Member

Thanks @haydentherapper.

LGTM, I'm just curious: is there any problem with the debian 10 static image? I always thought that being distroless makes the image distro-independent.

@loosebazooka
Copy link

There are some files in static that are pulled from debian repos like CA certs and such.

@loosebazooka
Copy link

loosebazooka commented Mar 26, 2024

also minor correction, the un-debian-versioned tag points to debian11, but we're going to start deprecating those images this year at some point.

so image:static (latest) would just start pointing to debian12 images

@nabokihms nabokihms merged commit 38cef0c into dexidp:master Mar 28, 2024
9 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants