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

change shared mounts for /var/lib/kubelet to rw #69

Closed
wants to merge 1 commit into from

Conversation

DorianGray
Copy link
Contributor

It turns out that on docker 1.10+, shared mounts and kubernetes don't play well together. Luckily, falling back to rw seems to fix it. I am not sure why hyperkube suggests a shared mount for this, but the cluster seems to work with it being rw instead. Again, this is on hypriotos.

kubernetes/kubernetes#18239

@DorianGray
Copy link
Contributor Author

It has something to do with namespace propagation, there is a PR in kubernetes to fix it but unfortunately it hasn't been merged yet.

@DorianGray
Copy link
Contributor Author

I just found the discussion in #40. would slave be a better mode?

@luxas
Copy link
Owner

luxas commented Mar 7, 2016

A shared mount is required, and it's the solution, not the problem as discussed in #40.
It doesn't work with slave or any other option.
See https://www.kernel.org/doc/Documentation/filesystems/sharedsubtree.txt why.

kubelet needs to mount tmpfses (service account certificates) and other volumes, although it isn't in the same namespace, therefore it needs a shared mount.

@DorianGray
Copy link
Contributor Author

weird, shared doesn't work on my setup...keeps saying that it's not a shared mount and causing hyperkube to get realmad.

@DorianGray
Copy link
Contributor Author

I'll do some more tinkering, see if I can figure it out.

@DorianGray DorianGray closed this Mar 7, 2016
@luxas
Copy link
Owner

luxas commented Mar 7, 2016

You'll have to set MountFlags= in docker.service and the dropins, then it'll work

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.

2 participants