Skip to content
This repository has been archived by the owner on Oct 1, 2020. It is now read-only.

Bump Kafka to version 0.9.0.0 #28

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

lagz0ne
Copy link

@lagz0ne lagz0ne commented Dec 22, 2015

No description provided.

@s7anley
Copy link

s7anley commented Jan 11, 2016

Hi @lagz0ne, have you tested it? Did you bump to any kind of problems with 0.9 version in this setup?

@lagz0ne
Copy link
Author

lagz0ne commented Jan 12, 2016

I used that for a while, didn't bump into any problem actually.
@flozano and I think @portenez has given it a tried, they may have more inside

@flozano
Copy link

flozano commented Jan 12, 2016

Works for us here, no problems

@portenez
Copy link

The image built this way works for basic scenarios. I haven't tried any more than the basics.

Also, I think I reverted to 8x, but I don't remember why. I can double check, but it might take a while.

@s7anley
Copy link

s7anley commented Jan 12, 2016

@portenez would be great if you could recall what was the reason 👍

@portenez
Copy link

Guys, I'm sorry I dig around and couldn't find a reason. I don't want you to keep waiting for me.

@akauppi
Copy link

akauppi commented Feb 16, 2016

Also interested in this, even if just as a branch.

Further notices:

  • the version of the Kafka within the Docker package could be clearly mentioned on the README. Now it has 0.8.2.1 but I needed to see inside the running container to find this out.
  • references to "Kafka 7" and "Kafka 8" in the README are vague

Disclaimer: I am not requiring "support" for this package. However, e.g. docker-it-scala, a library to use Docker containers from ScalaTest is referring to it.

@ijuma
Copy link

ijuma commented Feb 22, 2016

It would be good to bump the version to 0.9.0.1 which includes a number of fixes.

@arichiardi
Copy link

👍 for bumping

@gg7
Copy link

gg7 commented Jan 20, 2017

If you use ARG instead of ENV you wouldn't even have to edit the Dockerfile. See https://github.com/gg7/docker-kafka/blob/51be341802de622da29480b39998e530c27e08ef/Dockerfile#L8-L9

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants