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

Document allowing unsafe sysctls in minikube #5897

Closed
wants to merge 2 commits into from

Conversation

charlesverdad
Copy link
Contributor

@charlesverdad charlesverdad commented Oct 13, 2017

Add guide on allowing unsafe sysctls when using minikube


This change is Reviewable

@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 Oct 13, 2017
@k8sio-netlify-preview-bot
Copy link
Collaborator

k8sio-netlify-preview-bot commented Oct 13, 2017

Deploy preview ready!

Built with commit 144d579

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


```shell
$ minikube start --extra-config="kubelet.AllowedUnsafeSysctls=kernel.msg*,net.ipv4.route.min_pmtu"...
```
Only _namespaced_ sysctls can be enabled this way.
Copy link
Contributor

@steveperry-53 steveperry-53 Oct 13, 2017

Choose a reason for hiding this comment

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

This command is not rendering correctly in the preview. But I don't understand why. I'll investigate.

@steveperry-53
Copy link
Contributor

@vishh, @Random-Liu, @tallclair Can one of you give tech approval for this small change? Thanks.

Try to fix layout error in build preview
@charlesverdad
Copy link
Contributor Author

charlesverdad commented Oct 14, 2017

@steveperry-53 Because of the weird layout error in the developer preview, I'll close this in favor of #5894 , which renders correctly.

update: It looks like I had accidentally added leading spaces in each line of my commit here, causing the rendering issue. The PR in #5894 should be fine.

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