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

Support using the configured UUID instead of always generating a UUID… #133

Merged
merged 2 commits into from
Aug 24, 2016

Conversation

chirino
Copy link
Contributor

@chirino chirino commented Aug 24, 2016

… so that you can reuse IP addresses for hosts.

… so that you can reuse IP addresses for hosts.
chirino added a commit to chirino/minishift that referenced this pull request Aug 24, 2016
… you will get same IP address again.

Depends on the xhyve driver merging the following PR:
machine-drivers/docker-machine-driver-xhyve#133
@zchee
Copy link
Member

zchee commented Aug 24, 2016

@chirino Thanks for the pull request!

It seems LGTM, but now xhyve driver has not configure UUID flag.
Maybe this pull request for minikube or minishift?
I think better is also add uuid variable flag.

Thanks.

Edit: I means like --xhyve-uuid flag

@chirino
Copy link
Contributor Author

chirino commented Aug 24, 2016

Yep it's related to minikube/shift see: minishift/minishift#53

@zchee
Copy link
Member

zchee commented Aug 24, 2016

@chirino Thanks :)
LGTM.

@zchee
Copy link
Member

zchee commented Aug 24, 2016

@chirino I'm wait for ci result...

chirino added a commit to chirino/minishift that referenced this pull request Aug 24, 2016
… you will get same IP address again.

Depends on the xhyve driver merging the following PR:
machine-drivers/docker-machine-driver-xhyve#133
@zchee zchee merged commit b25ac3d into machine-drivers:master Aug 24, 2016
@zchee
Copy link
Member

zchee commented Aug 24, 2016

@chirino @jimmidyson merged. Thanks :)

@jimmidyson
Copy link

Awesome! Thanks for a great project btw 🍻

@zchee
Copy link
Member

zchee commented Aug 24, 2016

@jimmidyson yey \o/

@zchee
Copy link
Member

zchee commented Aug 24, 2016

@jimmidyson @dlorenc I think maybe kubenetes team and minikube/minishift like to use release version.
I will release the next version after some private works.

@jimmidyson
Copy link

We certainly do want to use it! Whenever you get a chance to release that would be great. Thanks!

r2d4 added a commit to r2d4/minikube that referenced this pull request Aug 24, 2016
r2d4 added a commit to r2d4/minikube that referenced this pull request Sep 19, 2016
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

3 participants