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

Update mount implementation, add mount integration tests, and check t… #1293

Merged
merged 1 commit into from Apr 6, 2017

Conversation

aaron-prindle
Copy link
Contributor

…hat path exists.

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Mar 24, 2017
@aaron-prindle aaron-prindle force-pushed the switch-mount branch 10 times, most recently from a51c017 to c4fb492 Compare March 27, 2017 17:40
@aaron-prindle
Copy link
Contributor Author

@minikube-bot retest this please

@dlorenc
Copy link
Contributor

dlorenc commented Mar 27, 2017

Were there any manual changes to the new go9p library?

@aaron-prindle aaron-prindle force-pushed the switch-mount branch 3 times, most recently from 821a9d3 to 63e002e Compare March 29, 2017 17:59
@codecov-io
Copy link

codecov-io commented Mar 29, 2017

Codecov Report

Merging #1293 into master will decrease coverage by 0.33%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #1293      +/-   ##
=========================================
- Coverage   43.43%   43.1%   -0.34%     
=========================================
  Files          48      48              
  Lines        2256    2269      +13     
=========================================
- Hits          980     978       -2     
- Misses       1117    1131      +14     
- Partials      159     160       +1
Impacted Files Coverage Δ
pkg/minikube/cluster/commands.go 61.79% <ø> (-3.68%) ⬇️
cmd/minikube/cmd/mount.go 5.26% <0%> (-1.19%) ⬇️
pkg/minikube/cluster/cluster.go 55% <0%> (-0.22%) ⬇️
pkg/minikube/kubeconfig/config.go 55.38% <0%> (-3.08%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c4911dc...8e801e3. Read the comment docs.

@aaron-prindle
Copy link
Contributor Author

Yes the "same" manual changes needed to be done as before in order to get the server working for Windows.

@aaron-prindle
Copy link
Contributor Author

@minikube-bot retest this please

@aaron-prindle
Copy link
Contributor Author

@minikube-bot retest this please

Copy link
Contributor

@r2d4 r2d4 left a comment

Choose a reason for hiding this comment

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

LGTM

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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants