-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Feature/dockerization #582
Conversation
On branch feature/dockerization Changes to be committed: modified: build.sbt modified: project/plugins.sbt
Changes to be committed: modified: build.sbt
can anyone take a look at this PR ? this PR will make docker building very simple. and version will be always matching version that's defined in sbt. |
do we have targeted release for this to be merged in? |
Neat! Any plan to make a docker hub repo for official releases? |
Good idea.
We can create an organization first.
R
On Wed, Dec 19, 2018 at 5:55 PM David Glasser ***@***.***> wrote:
Neat! Any plan to make a docker hub repo for official releases?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#582 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABEnE1kpUe_0vHZXOq29O2J0dBmfyPLrks5u6sPbgaJpZM4YdAYm>
.
--
*Best Regards*
*Ryne*
|
I've created a repo here: https://cloud.docker.com/u/kafkamanager/repository/list |
what would it entail? |
@patelh we will have an 'official dockerhub' for releasing official images for anyone who wants to deploy this kafka manager. versions will be preserved through image tags so ppl can use older versions if they wish. and also since this manager most of the time only needs |
I stumbled upon this repo on dockerhub. Thanks for putting efforts to release official images. Is it possible to add a |
@ziru we could, but someone need to maintain this docker repo and make sure the 'latest' is the latest. if you want I can make you as one of the owner so you can also maintain the latest tag anytime new version comes out. |
@linehrr Sounds good to me. I can help maintain the docker repo. |
@ziru right now it's manually, but I really like someone to add CI hook into github so anytime we have a new release tag, it can build automatically. |
@linehrr yup, auto push would be great. here is my dockerhub account: https://hub.docker.com/u/ziru |
@ziru added, you are owner and maintainer now. |
@linehrr thanks! done with publishing the 1.3.3.22 image with the latest tag to docker hub. |
I propose that we instead automate the build and push of release tags. |
@rajat404 it's integrated into sbt. take a look at build.sbt file. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
consider upgrading jdk version
val zipFile: File = dist.value | ||
|
||
new Dockerfile { | ||
from("openjdk:8-jre") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You are using jdk8 while the minimum requirements is 11+ in project readme
by running
sbt docker
, you will get a docker imagekafka-manager:<version>
locally.then by running
docker run -d -e<ENV VARS> kafka-manager:<version>
, you will have a running kafka-manager instance listening to port 9000.port could be mapped to any other port with
-p<any port>:9000
in docker run option.