-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Switch to the rkt iso by default #571
Comments
@dlorenc is the plan to move the code into this tree? |
Yeah, if @s-urbaniak agrees. |
@s-urbaniak have info on how to reproduce the OSX issues? |
@ryanj on virtualbox or vmware fusion? |
OK, so a few things:
@s-urbaniak so, I think you need to add some network drivers first to get this working |
@s-urbaniak I need vmxnet3 please as a kernel driver! |
@philips I added the driver in the minikube iso. Do you mind trying it out with https://github.com/coreos/minikube-iso/releases/download/v0.0.4/minikube-v0.0.4.iso? |
On OS X, using the new rkt v0.0.3 iso, |
@aecolley can you give us an example? |
the iso booted cleanly for me using Virtualbox 5.1 on OSX 10.10.2 and Virtualbox 5.0 on Fedora24. I didn't test the other providers, thanks for the additional info I successfully deployed an image from dockerhub (didn't test volume mounts), and was able to view the running container in the output of |
@aecolley can you give details on your OS, hypervisor, and version of minikube along with the example? |
@philips Certainly. Here's my test case script: #!/bin/bash
minikube stop || :
minikube delete || :
rm -fr mountable || :
set -o errexit
minikube start "$@"
dvm use 1.12.1
eval "$(minikube docker-env)"
mkdir mountable
touch mountable/proof
output="$(docker run --rm -v $PWD/mountable:/mounted:ro busybox:latest /bin/ls -asCF /mounted)"
printf 'Output: %s\n' "${output}"
if [[ "${output}" == *proof* ]]; then
echo PASS
exit 0
else
echo FAIL
exit 1
fi And here's an unedited copy-paste of the Terminal session in which I show it failing with the rkt iso and passing with the default setup. At the end, it shows the OS/virtualbox/minikube/docker versions as requested.
The version numbers are the same in both cases, but the other parts of the
Finally, it also fails when using the docker runtime with the rkt iso (i.e. with |
@s-urbaniak put up a new version of the ISO which WFM! https://github.com/coreos/minikube-iso/releases/download/v0.0.4/minikube-v0.0.4.iso Confirmed working with minikube v0.10.0 and OSX and vmware fusion. Thanks @s-urbaniak |
@s-urbaniak what is the next step to move to this ISO by default? |
@philips I can prepare a PR against minikube in the next week, I am still working on xhyve support though: https://github.com/coreos/minikube-iso/issues/17 @dlorenc Do you see xhyve support as a blocker right now or do you think I can proceed with a "boot2docker to minikube-iso" PR? |
I personally don't see xhyve as a blocker as it is still really hard to get that working. We need to figure out how to bundle it instead of relying on the libmachine external driver, etc. |
I suppose we could make this iso default automatically for people passing the container-runtime=rkt flag, but there are quite a few xhyve users that I'd rather not break. I think the xhyve issues are fixed now though, I'll do some final testing tomorrow. @s-urbaniak do you want to send a PR to start moving the ISO build scripts from your repo here? |
@dlorenc @s-urbaniak mentioned he got xhyve working yesterday after I commented. |
@dlorenc working on the changes to pull the ISO build scripts here right now. Not sure if I'll make it today though. xhyve basically works according to https://github.com/coreos/minikube-iso/issues/17#issuecomment-250664776, @zchee is trying to push this upstream via moby/hyperkit#63. |
@dlorenc @philips @s-urbaniak If there is no response from docker-dev until merge machine-drivers/docker-machine-driver-xhyve#138 FYI, I'm already using the zchee/libhyperkit that the static vendoring hyperkit and other patched(cgo problem, dispatch_loop(Apple GCD), lib9p server), so we do not need to wait for issue response. How you do you feel about it? |
@zchee I'd say go with it :-) |
+1, that sounds great to me. |
@s-urbaniak @dlorenc @philips Done :) I was tested and successful with
If you want to pre-compile, Here is |
I suggest to reopen this, as we have the scripts now available to generate the ISO, but inside minikube we are still pointing to the boot2docker based iso image as per [1]. @dlorenc @r2d4 I don't see pointers in the source on how the upload is happening on https://storage.googleapis.com/minikube. Please let me know if there is anything on how I can help to make this happen, so we can shut down https://github.com/coreos/minikube-iso/releases :-) [1]
|
I reopened this, I think it got auto-closed from the PR. Right now, we are just pushing the iso with gsutil https://github.com/kubernetes/minikube/tree/master/deploy/iso/boot2docker I can push the previous releases of coreos/minikube-iso to our storage bucket. I'll let you know when its done. Another thing we might need to figure are names for the ISOs - the b2d one is currently called |
minikube-alpha.iso sgtm for the time being. |
I think we're basically ready to make this default. Any concerns about doing this in 0.13? |
Ref #168 #570
The text was updated successfully, but these errors were encountered: