Skip to content

Conversation

@ianks
Copy link
Contributor

@ianks ianks commented Sep 15, 2014

This will reduce some of the bloat from Ubuntu, yet give us the apt-get goodness. From my initial tests the image size is smaller, and things seems to be functioning as expected.

Size of Ubuntu Image: 1.36gb
Size of Debian Image: 673mb

@summera
Copy link

summera commented Sep 15, 2014

👍

@pblittle pblittle self-assigned this Sep 15, 2014
@pblittle
Copy link
Owner

@ianks, I think this is a great idea to discus. I have explored a few different options so far, from Debian Wheezy to CoreOs.

My primary reservation for not pulling the trigger yet is the lack of a solid test suite. I would feel much more comfortable making low level changes like this with solid tests in place first.

I have a decent embedded elasticsearch test now, but I don't have a linked container test that I'm happy with.

Also, if the base OS is changed, I might move back to using an updated version of my docker-base image. That will slim this Dockerfile down and cache the base image for other projects.

@ianks, @summera, what are your thoughts?

@ianks
Copy link
Contributor Author

ianks commented Sep 16, 2014

From what I can tell from the code, this image is not reliant on any Ubuntu specific features. It is booting elasticsearch, wget and using some typical unix commands. None of which have Ubuntu OS features as a dependency. The fact that init is never being called means you are essentially using Ubuntu for its package manager.

@pblittle
Copy link
Owner

@ianks, you're correct about the lack of reliance on Ubuntu OS features. Thanks for pointing that out and the contribution.

@ianks
Copy link
Contributor Author

ianks commented Sep 17, 2014

Sure, this image has been useful :)

@ianks
Copy link
Contributor Author

ianks commented Sep 18, 2014

@pblittle What are your thoughts on merging this?

@pblittle
Copy link
Owner

@ianks, I was traveling this week and am finally getting caught up tonight. I'll pull it in as soon as the DEBIAN_FRONTEND discussion in #23 is sorted out. I would like to bundle both features in the rel-0.8.0 release.

pblittle pushed a commit that referenced this pull request Sep 19, 2014
Use Debian instaead of Ubuntu as base
@pblittle pblittle merged commit 79c357e into pblittle:develop Sep 19, 2014
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.

3 participants