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

kubeadm iptables settings on centos #4340

Closed
wants to merge 76 commits into from
Closed

kubeadm iptables settings on centos #4340

wants to merge 76 commits into from

Conversation

fabriziopandini
Copy link
Member

@fabriziopandini fabriziopandini commented Jul 10, 2017

when running kubeadm on centos pre-fligth check fails due to

[preflight] Some fatal errors occurred:
	/proc/sys/net/bridge/bridge-nf-call-iptables contents are not set to 1

this PR contains description of required step to fix this issue (solution was described in previous version of kubeadm documentation, reported here with few little improvements)

After updating docs, IMO we can close kubernetes/kubeadm#312


This change is Reviewable

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Jul 10, 2017
@chenopis
Copy link
Contributor

chenopis commented Jul 10, 2017

Deploy preview ready!

Built with commit 3cf1942

https://deploy-preview-4340--kubernetes-io-vnext-staging.netlify.com/

Copy link
Contributor

@chenopis chenopis left a comment

Choose a reason for hiding this comment

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

Some style and markdown formatting issues.

Note: Disabling SELinux by running `setenforce 0` is required to allow
containers to access the host filesystem, which is required by pod networks for
example. You have to do this until SELinux support is improved in the kubelet.
Note:
Copy link
Contributor

Choose a reason for hiding this comment

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

This should be in bold, so: **Note:**

containers to access the host filesystem, which is required by pod networks for
example. You have to do this until SELinux support is improved in the kubelet.
Note:
- Disabling SELinux by running `setenforce 0` is required to allow containers to access the host filesystem, which is required by pod networks for example. You have to do this until SELinux support is improved in the kubelet.
Copy link
Contributor

@chenopis chenopis Jul 10, 2017

Choose a reason for hiding this comment

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

Add a newline before the unordered list so that the markdown is rendered correctly. You can preview it here: https://deploy-preview-4340--kubernetes-io-vnext-staging.netlify.com/docs/setup/independent/install-kubeadm/#installing-kubelet-and-kubeadm

Note:
- Disabling SELinux by running `setenforce 0` is required to allow containers to access the host filesystem, which is required by pod networks for example. You have to do this until SELinux support is improved in the kubelet.
- Some users on RHEL/CentOS 7 have reported issues with traffic being routed incorrectly due to iptables being bypassed. You should ensure `net.bridge.bridge-nf-call-iptables` is set to 1 in your `sysctl` config, e.g.
``` bash
Copy link
Contributor

Choose a reason for hiding this comment

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

Add a newline before the code block so that it renders correctly. See the preview.

Chao Xu and others added 18 commits July 27, 2017 15:35
Deprecate system:nodes binding
…hub.io into release-1.8

* 'master' of https://github.com/kubernetes/kubernetes.github.io:
  Add noindex to vnext-staging
  Add note for 1.8 release. (#4632)
  fix create-cluster-kubeadm docs
  Update aws.md
  add newline above kube-apiserver
  Update scheduling-gpus.md
  Fix spacing with shell commands
  Documenting another (potential) callout issue (#4604)
  Update reference docs with most recent version of brodocs (#4600)
  Update kubernetes-api.md
  fix spelling mistake
  fix typos in fedration docs
  fix kubelet garbage collected typo
  fix typo in network-policies.md
  fix typo
  Add /_includes/glossary directory for snippets (#4628)
  Fixed typo in getting started guide using minikube
  Update to include Python library (#4624)
  Rename OWNERS assignees: to approvers:
…hub.io into release-1.8

* 'master' of https://github.com/kubernetes/kubernetes.github.io:
  fix noindex for vnext-staging (#4640)
  Add network overlay details to cloud routes flag
  Revert "Update volumes.md"
  add examples for interpod configurations (#4557)
  Update "readonly" in abac
[1.8] StatefulSet `initialized` annotation is now ignored.
mhausenblas and others added 12 commits September 14, 2017 14:18
I updated this doc yesterday and I was absolutely sure I fixed this, but I just saw that this commit got lost somehow.

This was introduced recently in #5440.
…hub.io into release-1.8

* 'master' of https://github.com/kubernetes/kubernetes.github.io:
  ZTE-SH-CN-debug-pod-replication-controller-2017-09-14-14
  ZTE-SH-CN-define-command-argument-container (#5381)
  Update index.md
  Update Kompose docs
  Improve host aliases page (#5443)
  Add link to AlwaysPullImages admission controller
  Revert "create homepage for user journeys"
  create homepage for user journeys
  fix the command output
  ZTE-SH-CN-run-application-run-single-instance-stateful-application-pr-2017-… (#5363)
  Add a note to static pod manifest scanning
  revert WordPress and MySQL PV doc changes to use apps/v1beta2 APIs (#5461)
  Fix pod probes yaml file
  Redirect v1 docs. (#5460)
  Fix error link (#5459)
  Fix typo in deployment documentation

# Conflicts:
#	docs/tutorials/stateful-application/basic-stateful-set.md
@chenopis
Copy link
Contributor

@fabriziopandini I don't think this will make it into the 1.8 release. We can leave it on the master branch and merge it in after the release is out. You will need to fix the merge conflict and get a Tech LGTM though.

@fabriziopandini
Copy link
Member Author

@chenopis Thanks for the update!

@chenopis
Copy link
Contributor

@janetkuo @luxas Can either of you do a quick tech review for this? Thx

Copy link
Member

@luxas luxas left a comment

Choose a reason for hiding this comment

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

/lgtm
Needs rebase @fabriziopandini

@chenopis
Copy link
Contributor

@fabriziopandini Can you fix the merge conflict?

@k8s-ci-robot k8s-ci-robot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Sep 30, 2017
@fabriziopandini
Copy link
Member Author

Sorry screw up rebasing :-( . I Close this and send a new one...

@k8sio-netlify-preview-bot
Copy link
Collaborator

Deploy preview ready!

Built with commit 483b438

https://deploy-preview-4340--kubernetes-io-master-staging.netlify.com

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.

improve kubeadm preflight message for bridge-nf-call-iptables