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

Add new VMware unified driver to supported list #2 #3534

Merged
merged 2 commits into from
Jan 17, 2019
Merged

Add new VMware unified driver to supported list #2 #3534

merged 2 commits into from
Jan 17, 2019

Conversation

frapposelli
Copy link
Member

This is a follow up from #2606 as the original fork was removed and unable to update the PR

This PR adds the new VMware unified driver to the supported list, the driver now supports Fusion and Workstation across Windows, Linux and macOS.

A deprecation note has been added for the bundled vmwarefusion driver.

Ref: #2118 #2526 machine-drivers/discussion#5 machine-drivers/docker-machine-driver-vmware#1

Fixes: machine-drivers/docker-machine-driver-vmware#1 #2118

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Jan 16, 2019
@k8s-ci-robot k8s-ci-robot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Jan 16, 2019
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: frapposelli
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: luxas

If they are not already assigned, you can assign the PR to them by writing /assign @luxas in a comment when ready.

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@minikube-bot
Copy link
Collaborator

Can one of the admins verify this patch?

@frapposelli
Copy link
Member Author

/assign @tstromberg

@@ -176,6 +177,22 @@
pruneopts = "NUT"
revision = "3165313d6d3f973ec0b0ed3ec5a63b520e065d40"

[[projects]]
digest = "1:63ede27834b468648817fb80cfb95d40abfc61341f89cb7a0d6779b6aa955425"
name = "github.com/google/go-github"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any idea what is pulling in this dependency? It seems unusual to include a github client into minikube.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it looked weird to me as well, and I honestly don't know where it's coming from.

Before submitting the PR I double checked and it is not coming from our github.com/machine-drivers/docker-machine-driver-vmware import, so I assumed that the dep ensure command pulled a dependency coming from somewhere else in the minikube codebase.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can confirm that running dep ensure causes these to appear on my end. Strange!

@tstromberg
Copy link
Contributor

@minikube-bot OK to test

@mlebrun
Copy link

mlebrun commented Jan 28, 2019

@frapposelli I updated Minikube to the 0.33.1 release to try out this driver and it seems like VMware Tools isn't installed? Which then makes Shared Folder support a bit unstable/unusable. I wasn't sure if there's anything I can do to get better Shared Folder support from Fusion (OSX) within Minikube? (I realize you're not full time Minikube maintainer, seeing if you have any input!) The minikube --mount method isn't that great when mounting a lot of files (or maybe just seems better suited to smaller projects?) so that's why I'm hoping for better Shared Folder support. Wasn't sure if you give any insight into that or not?

@mlebrun
Copy link

mlebrun commented Jan 28, 2019

Seeing this which I just noticed when running verbose logs on minikube start:

(minikube) DBG | executing: /Applications/VMware Fusion.app/Contents/Public/vmrun -gu docker -gp tcuser addSharedFolder /Users/{user}/.minikube/machines/minikube/minikube.vmx Users /Users
(minikube) DBG | executing: /Applications/VMware Fusion.app/Contents/Public/vmrun -gu docker -gp tcuser runScriptInGuest /Users/{user}/.minikube/machines/minikube/minikube.vmx /bin/sh [ ! -d /hosthome ]&& sudo mkdir /hosthome; sudo mount --bind /mnt/hgfs//hosthome /hosthome || [ -f /usr/local/bin/vmhgfs-fuse ]&& sudo /usr/local/bin/vmhgfs-fuse -o allow_other .host:/Users /hosthome || sudo mount -t vmhgfs -o uid=$(id -u),gid=$(id -g) .host:/Users /hosthome
(minikube) DBG | Guest program exited with non-zero exit code: 1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants