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

Remove distroless image generation #22583

Merged
merged 1 commit into from
Jan 3, 2022

Conversation

cescoffier
Copy link
Member

Fixes #21557.

Remove the distroless docker file generation in favor of micro.

@quarkus-bot quarkus-bot bot added area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/documentation area/platform Issues related to definition and interaction with Quarkus Platform labels Jan 3, 2022
Quarkus provides the `quay.io/quarkus/quarkus-distroless-image:1.0` image.
It contains the required packages to run a native executable and is only **9Mb**.
Just add your application on top of this image, and you will get a tiny container image.

Distroless images should not be used in production without rigorous testing.
Copy link
Member

Choose a reason for hiding this comment

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

Wondering if we should remove this altogether and stop building the image?

Copy link
Member Author

Choose a reason for hiding this comment

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

We had requests to have one. It can still be useful even if you get a similar size (15Mb overhead) with the micro image.

Copy link
Contributor

@gastaldi gastaldi left a comment

Choose a reason for hiding this comment

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

LGTM

@gsmet gsmet merged commit 77d228b into quarkusio:main Jan 3, 2022
@quarkus-bot quarkus-bot bot added this to the 2.7 - main milestone Jan 3, 2022
@cescoffier cescoffier deleted the remove-distroless branch January 3, 2022 17:47
@flauschie
Copy link

Unless I'm mistaken, distroless is no option for AWS Lambda?

The Quarkus lambda guide is also pulling public.ecr.aws/lambda/provided or public.ecr.aws/lambda/java:11

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/documentation area/platform Issues related to definition and interaction with Quarkus Platform
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Switch from distroless to micro
5 participants