Skip to content

Dockerfile - #295

Open
n0mer wants to merge 3 commits into
yahoo:masterfrom
n0mer:master
Open

Dockerfile #295
n0mer wants to merge 3 commits into
yahoo:masterfrom
n0mer:master

Conversation

@n0mer

@n0mer n0mer commented Sep 9, 2016

Copy link
Copy Markdown

Sample Dockerfile

@yahoocla

yahoocla commented Sep 9, 2016

Copy link
Copy Markdown

Thank you for submitting this pull request, however I do not see a valid CLA on file for you. Before we can merge this request please visit https://yahoocla.herokuapp.com/ and agree to the terms. Thanks! 😄

@yahoocla

yahoocla commented Sep 9, 2016

Copy link
Copy Markdown

CLA is valid!

@clehene

clehene commented Sep 30, 2016

Copy link
Copy Markdown

This will generate a rather large container.
There's no need to keep the build results in the container.
I'd also recommend a smaller base image like https://github.com/anapsix/docker-alpine-java

@dbeckham

Copy link
Copy Markdown

I've been using this image very successfully for some time now: https://hub.docker.com/r/sheepkiller/kafka-manager/

It allows for ENV variable based configuration, is generally very stable, and has an alpine based option.

@hleb-albau

hleb-albau commented Oct 2, 2017

Copy link
Copy Markdown

@linehrr

linehrr commented Nov 21, 2018

Copy link
Copy Markdown

I've made a docker build embedded into sbt by sbt docker plugin, so sbt docker will build the image on local. please refer to this PR: #582

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.

6 participants