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

use FROM fedora-minimal instead of centos:7 #9

Closed
wants to merge 1 commit into from
Closed

use FROM fedora-minimal instead of centos:7 #9

wants to merge 1 commit into from

Conversation

vorburger
Copy link
Contributor

as that creates a significantly (100 MB) smaller container image

as that creates a significantly (100 MB) smaller container image
@vorburger
Copy link
Contributor Author

vorburger commented Dec 17, 2018

Don't know how you guys feel about this idea?

If acceptable, docs/ would also have to be updated accordingly.

FTR: There is also a registry.centos.org/centos/centos7-atomic which from what little I gather has a similar purpose and roughly (~ 10M diff) size, but is 7 months older; if anyone better understands which of these (community) images is the one we could be promoting here instead of the "full blown" centos one, please shout.

PS: For an exploration into possible future even much smaller images, see quarkusio/quarkus#326

@cescoffier
Copy link
Member

@n1hility as you did the docker file we use, what do you think?

@cescoffier
Copy link
Member

Before merging this:

[ ] The getting started guide needs to be updated
[ ] The generated docker file (created by the create project) needs to be updated

@cescoffier
Copy link
Member

I believe we should move forward on this and open a PR on shamrock directly.

@cescoffier
Copy link
Member

I will do that Today.

@cescoffier
Copy link
Member

The PR has been merged in the development branch.

@cescoffier cescoffier closed this Dec 19, 2018
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.

2 participants