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

Remove --skip-preflight-checks, ignore unwanted checks #2403

Merged
merged 1 commit into from
Feb 27, 2018

Conversation

afbjorklund
Copy link
Collaborator

@afbjorklund afbjorklund commented Jan 7, 2018

Better to ignore the checks that we don't want, rather than
disabling all of them in case something important appears...

Currently known failing checks:
[WARNING SystemVerification]: docker version is greater than
the most recently validated version. Docker version: 17.06.0-ce. Max validated version: 17.03
[WARNING FileExisting-crictl]: crictl not found in system path
[WARNING Service-Docker]: docker service is not enabled, please run 'systemctl enable docker.service'
[ERROR DirAvailable--etc-kubernetes-manifests]: /etc/kubernetes/manifests is not empty
[ERROR Swap]: running with swap on is not supported. Please disable swap


https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG-1.9.md

The kubeadm --skip-preflight-checks flag is now deprecated and will be removed in a future release.

Better to ignore the checks that we don't want, rather than
disabling all of them in case something important appears...

Currently known failing checks:
	[WARNING SystemVerification]: docker version is greater than
the most recently validated version. Docker version: 17.06.0-ce. Max validated version: 17.03
	[WARNING FileExisting-crictl]: crictl not found in system path
	[WARNING Service-Docker]: docker service is not enabled, please run 'systemctl enable docker.service'
	[ERROR DirAvailable--etc-kubernetes-manifests]: /etc/kubernetes/manifests is not empty
	[ERROR Swap]: running with swap on is not supported. Please disable swap
@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Jan 7, 2018
@afbjorklund
Copy link
Collaborator Author

crictl was added as: #2404

@minikube-bot
Copy link
Collaborator

Can one of the admins verify this patch?

@dlorenc
Copy link
Contributor

dlorenc commented Jan 8, 2018

@minikube-bot ok to test

@afbjorklund
Copy link
Collaborator Author

afbjorklund commented Jan 8, 2018

The systemd files were removed in 56e250e, since it needs runtime configuration. But it could probably "enable" the needed runtime, if that makes kubeadm happy.

sudo systemctl daemon-reload &&
sudo systemctl enable kubelet &&
sudo systemctl enable $RUNTIME &&
sudo systemctl start kubelet

Where $RUNTIME would be the same as kubelet.service, maybe docker.service instead of docker.socket:

[Install]
{{if or (eq .ContainerRuntime "cri-o") (eq .ContainerRuntime "cri")}}Wants=crio.service{{else}}Wants=docker.socket{{end}}

Since: docker.socket depends on docker.service, but kubeadm only checks docker.service.

@afbjorklund
Copy link
Collaborator Author

Swap is discussed in #1827

@afbjorklund
Copy link
Collaborator Author

Service enabled in: #2406

@afbjorklund
Copy link
Collaborator Author

1.9.1 says:
Flag --skip-preflight-checks has been deprecated, it is now equivalent to --ignore-preflight-errors=all

@dlorenc
Copy link
Contributor

dlorenc commented Feb 20, 2018

Is this ready to be merged?

@afbjorklund
Copy link
Collaborator Author

@dlorenc : I think so, it should handle all errors. The warnings however, are addressed in separate PRs

@dlorenc
Copy link
Contributor

dlorenc commented Feb 26, 2018

@minikube-bot test this please

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/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants