noony / prometheus-solr-exporter

Solr exporter for prometheus.
Apache License 2.0
27 stars 18 forks source link

Update Dockerfile #9

Closed koep closed 7 years ago

koep commented 7 years ago

I want to use your solr exporter in my Kubernetes setup (for simplicity I want my k8s cluster to pull the image from Docker Hub).

For this I updated the Dockerfile and created an automated build (https://hub.docker.com/r/koep/prometheus-solr-exporter/).

In case you want to pull this we should update the README.md accordingly. Otherwise just close this PR :)

noony commented 7 years ago

I already had a docker hub image : https://hub.docker.com/r/noony/prometheus-solr-exporter/

koep commented 7 years ago

Oh sorry I missed that it's already on docker hub (maybe update the docs?).

I guess the only difference is that with the new Dockerfile the build happens inside a docker container. This way the build can be automated on docker hub (meaning you don't have to login to docker hub and push the image) and you don't need Golang on your host.

Also you don't have to pull a "binary blob" from docker hub, but you can see the actual dockerfile and reproduce it yourself. https://docs.docker.com/docker-hub/builds/

noony commented 7 years ago

Yes ok I see the difference, thanks, I will merge this PR and think about put in place automated build and update the documentation 👍 .

koep commented 7 years ago

Also the build happens on every git commit push. I think you can even configure docker tags to match github tags etc. Replacing it will be pretty straight forward I think!

noony commented 7 years ago

Yes, Thanks ! Tonight I will put that in place, thanks

noony commented 7 years ago

I close this PR. Thanks I prefer to not automatically build for now. I've added the link to docker image in README.md