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

Error: mkdir: cannot create directory ‘/var/lib/kubelet’ #775

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

stanfordpeng
Copy link

Error:
mkdir: cannot create directory ‘/var/lib/kubelet’

It has been created in the previous section:
https://github.com/kelseyhightower/kubernetes-the-hard-way/blob/master/docs/04-certificate-authority.md#distribute-the-client-and-server-certificates

@zatricky
Copy link

zatricky commented May 24, 2024

If following the steps in order then it would have been done correctly in 04 and there would be no error.

This change would make it so the user doesn't have to read and understand what they are doing, putting it closer to simply being a fire-and-forget deploy script, which is counter to my understanding of the aim of the repo. :-|

My suggestion if you insist on a change would be to put a comment along the lines of "Note that the /var/lib/kubelet folder was created on each node in a previous step in 04".

@jasonwashburn
Copy link

If following the steps in order then it would have been done correctly in 04 and there would be no error.

This change would make it so the user doesn't have to read and understand what they are doing, putting it closer to simply being a fire-and-forget deploy script, which is counter to my understanding of the aim of the repo. :-|

That's a fair take as well...fixing it either way (removing it, or adding -p) makes sense to me. I would lean against leaving it as is, knowing the error will get thrown though.

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.

4 participants