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

Version up mesos to 1.3.0 #312

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

fuji-151a
Copy link
Contributor

Hi, I used mesos-kafka in mesos version 1.3.0.
However, it was not started up because it was built mesos version 0.28.0.
So, I update mesos version to 1.3.0.

I checked running in mesos version 1.3.0.

Please this pull request.
Thanks

@steveniemitz
Copy link
Contributor

What error did you get when starting up? Would going to 1.0.0 be enough to fix it? Otherwise, what's the minimum master version the 1.3.0 driver can support?

@fuji-151a
Copy link
Contributor Author

fuji-151a commented Sep 13, 2017

@steveniemitz Thank you reviewing!!

What error did you get when starting up? Would going to 1.0.0 be enough to fix it?

Oh...Sorry. I forgot to store the error log.
But, when I used mesos kafka building in mesos 0.28, mesos agent didn't boot kafka broker.

what's the minimum master version the 1.3.0 driver can support?

I verified that it is running in version 1.1.0 of mesos master/agent
So, I think that it will be running in version 1.x.x . maybe...

@fuji-151a fuji-151a closed this Sep 22, 2017
@fuji-151a fuji-151a reopened this Sep 22, 2017
@fuji-151a fuji-151a closed this Dec 2, 2017
@fuji-151a fuji-151a reopened this Dec 2, 2017
@steveniemitz
Copy link
Contributor

steveniemitz commented Dec 14, 2017

I think this is fine at this point. I'll add a note that the driver version can be changed if needed. This breaks a lot of the unit tests though, can you look into those?

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.

2 participants