-
Notifications
You must be signed in to change notification settings - Fork 34
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
docker-machine tweaks in support of PR #13 #14
base: master
Are you sure you want to change the base?
Conversation
Search for boot files using patterns instead of hard coded paths. Similar to the docker-machine-driver-xhyve approach. Signed-off-by: John Coyle <[email protected]>
Only set CPU or memory values when requested value is greater than the default. Signed-off-by: John Coyle <[email protected]>
Hyperkit uses this UUID to assign a MAC address that the driver is expecting. The driver searches for this MAC address in the DHCP lease file to resolve the machine. Using a UUID based on machine name minimizes entries in the DHCP lease file. It also provides IP address consistency while the DHCP lease is valid. Signed-off-by: John Coyle <[email protected]>
Set a default disk size. Move init of BaseDriver.SSHUser to Create() to avoid overwrite. Signed-off-by: John Coyle <[email protected]>
…-count, memory-size). Signed-off-by: John Coyle <[email protected]>
This has been tested and works with minikube as well. I'm would be happy for this to get merged and to abandon my PR. |
Can we merge this? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR is really important to merge in. Without these changes it is impossible to use the project. cc @r2d4
if d.CPU > defaultCPUs { | ||
h.CPUs = d.CPU | ||
} | ||
if d.Memory > defaultMemory { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if d.Memory > defaultMemory { | |
if d.Memory > 0 { | |
// 0 or minMemory, if known |
While unlikely and against better judgement, user should be allowed to lower the request for memory. Same could be done for CPU count, if fractions of CPU are supported by hyperkit.
Any chance this will be merged? |
Can verify that this now makes docker-machine create --driver hyperkit usable on MacOS :) |
Hi @zazula I got the error:
|
Try adding this to your docker-machine command: --engine-env DOCKER_RAMDISK=true |
Strange, but with this env I even can't pull image, btw it works normally on vbox machine:
|
Try with my clone of the repo here: https://github.com/zazula/docker-machine-driver-hyperkit
then
|
Also, make sure your |
It looks like it was trouble with my host /etc/resolv.conf, so docker-machine couldn't access internet. At this time I was able to run containers 🎉 Anyway thx! Now I got significant increase mount host filesystem performance 🥳 |
Great! I don't recall doing anything with the CGO_ENABLED flag but glad you have it working. I'm working on some fixes for NFS mounts and exposing some of the NFS config flags as --hyperkit-nfs-xxx |
hi, can someone provide an update on this ? |
Depends on #13
Tested against docker-machine.