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

[WIP] Build docker image from ruby:2.3-alpine #1695

Closed
wants to merge 1 commit into from

Conversation

adammcmaster
Copy link
Contributor

This changes the Dockerfile to build FROM ruby:2.3-alpine -- there's really no reason for us to build our own ruby image when there's an official one, and the -alpine version is tiny. This reduces the final panoptes image size to 130 MB (down from the current 248 MB).

This will need a thorough test when it's deployed to staging. Alpine uses musl libc rather than glibc, so we'll need to make sure this doesn't have any unintended effects.

@camallen
Copy link
Contributor

camallen commented Mar 2, 2016

can we move this to a staging docker file that we can use for the staging infrastructure via the dockerfile: Dockerfile.staging cmd? then we can get this out and running to see how it performs.

@adammcmaster adammcmaster force-pushed the ruby-alpine branch 4 times, most recently from f549124 to 0a6fb38 Compare March 2, 2016 13:55
@adammcmaster adammcmaster changed the title Build docker image from ruby:2.3-alpine [WIPBuild docker image from ruby:2.3-alpine Mar 2, 2016
@adammcmaster adammcmaster changed the title [WIPBuild docker image from ruby:2.3-alpine [WIP] Build docker image from ruby:2.3-alpine Mar 2, 2016
@adammcmaster
Copy link
Contributor Author

After testing it looks like this will depend on rubyjs/therubyracer#378 unless we can find a workaround.

@camallen
Copy link
Contributor

closing this for now - that linked issue is pretty stale. Let's re-open when we look at this again.

@camallen camallen closed this May 17, 2016
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.

2 participants