Skip to content

Conversation

@paguos
Copy link
Contributor

@paguos paguos commented Nov 14, 2025

Description of your changes

Fixes #

I have:

@paguos
Copy link
Contributor Author

paguos commented Nov 14, 2025

Hey crossplane team! I noticed that for some reason this image is using a different base image (example).

Copy link
Member

@jbw976 jbw976 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for noticing this and submitting a fix @paguos - what do you think about making it static-debian12:nonroot to completely match other functions?

Dockerfile Outdated
# Produce the Function image. We use a very lightweight 'distroless' image that
# does not include any of the build tools used in previous stages.
FROM gcr.io/distroless/base-debian12 AS image
FROM gcr.io/distroless/base-debian12:nonroot AS image
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes that makes sense, I updated the image. Can you have another look, please?

@paguos paguos changed the title fix(Dockerfile): use base-debian12 nonroot image fix(Dockerfile): use static-debian12 nonroot image Nov 17, 2025
Copy link
Member

@jbw976 jbw976 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

awesome, thank you for making this update!

@dalton-hill-0 i'm not a maintainer on this repo, so feel free to put your final approval on too 😉

Copy link
Collaborator

@dalton-hill-0 dalton-hill-0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for adding this 🙂

@dalton-hill-0 dalton-hill-0 merged commit 658cc48 into crossplane-contrib:main Nov 17, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants