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

Feature/dockerization #582

Merged
merged 2 commits into from
Dec 17, 2018
Merged

Feature/dockerization #582

merged 2 commits into from
Dec 17, 2018

Conversation

linehrr
Copy link

@linehrr linehrr commented Nov 14, 2018

by running sbt docker, you will get a docker image kafka-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.

ryne.yang added 2 commits November 13, 2018 19:13
 On branch feature/dockerization
 Changes to be committed:
	modified:   build.sbt
	modified:   project/plugins.sbt
 Changes to be committed:
	modified:   build.sbt
@linehrr linehrr mentioned this pull request Nov 21, 2018
@linehrr
Copy link
Author

linehrr commented Nov 26, 2018

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.

@linehrr
Copy link
Author

linehrr commented Dec 7, 2018

do we have targeted release for this to be merged in?

@patelh patelh merged commit 4ce3f60 into yahoo:master Dec 17, 2018
@glasser
Copy link

glasser commented Dec 19, 2018

Neat! Any plan to make a docker hub repo for official releases?

@linehrr
Copy link
Author

linehrr commented Dec 19, 2018 via email

@linehrr
Copy link
Author

linehrr commented Dec 24, 2018

I've created a repo here: https://cloud.docker.com/u/kafkamanager/repository/list
I can transfer the ownership to anyone who's willing to maintain this repo.

@patelh
Copy link
Collaborator

patelh commented Dec 24, 2018

what would it entail?

@linehrr
Copy link
Author

linehrr commented Dec 24, 2018

@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 $ZK_HOSTS env var to be injected, this makes it perfect to fit in containerization and to be env agnostic plus all meta data writes into zookeeper, no local storage needed, which also makes it ephemeral and plays nicely inside container orchestration frameworks.
therefore I think it would be a great idea to make an official docker hub repo for this project.
what do you guys think ?

@ziru
Copy link

ziru commented Jan 3, 2019

I stumbled upon this repo on dockerhub.
https://hub.docker.com/r/kafkamanager/kafka-manager/tags

Thanks for putting efforts to release official images. Is it possible to add a latest tag?

@linehrr
Copy link
Author

linehrr commented Jan 3, 2019

@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.

@ziru
Copy link

ziru commented Jan 4, 2019

@linehrr Sounds good to me. I can help maintain the docker repo.
How did you publish the image to docker hub? Manually?

@linehrr
Copy link
Author

linehrr commented Jan 4, 2019

@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.
please give me your dockerhub username so I can add you as owner.

@ziru
Copy link

ziru commented Jan 4, 2019

@linehrr yup, auto push would be great. here is my dockerhub account: https://hub.docker.com/u/ziru

@linehrr
Copy link
Author

linehrr commented Jan 4, 2019

@ziru added, you are owner and maintainer now.

@ziru
Copy link

ziru commented Jan 5, 2019

@linehrr thanks! done with publishing the 1.3.3.22 image with the latest tag to docker hub.

@joekohlsdorf
Copy link

I propose that we instead automate the build and push of release tags.
CircleCI for example is free for open source projects: https://circleci.com/docs/2.0/oss/

@rajat404
Copy link

@linehrr @ziru Can you please point me to the Dockerfile. Wanted to read the code.
Thanks!

@linehrr
Copy link
Author

linehrr commented Aug 20, 2019

@rajat404 it's integrated into sbt. take a look at build.sbt file.

Copy link

@metao1 metao1 left a 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")
Copy link

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

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.

None yet

8 participants