website
website copied to clipboard
Improve "Reserve Compute Resources for System Daemons" doc
- ๐ Add a link to how to change kubelet settings
- ๐๏ธ Remove mention of k8s 1.17
- ๐งน Remove deprecated CLI flags and replace by KubeletConfiguration settings ; and properly document defaults
- ๐ properly document enforcement (
enforceNodeAllocatable
should be set)
Pull request preview available for checking
Built without sensitive environment variables
Name | Link |
---|---|
Latest commit | 2acc4f72c74326838208831d10531d96a5d92e7a |
Latest deploy log | https://app.netlify.com/sites/kubernetes-io-main-staging/deploys/665e23dd23ccba00086fc7d8 |
Deploy Preview | https://deploy-preview-45771--kubernetes-io-main-staging.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
Appart from the included changes, I find the Kube Reserved
and System Reserved
section misleading.
I first thought that those would map to an amount of reserved memory and cpu (i.e. requests), while the current behavior is to set limits.
Thanks @tengqm. I've applied your suggestions.
/label tide/merge-method-squash
@tengqm How can we move this PR forward?
@sathieu Please check the feedback about headings and fix it if possible.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: tengqm
The full list of commands accepted by this bot can be found here.
The pull request process is described here
- ~~content/en/docs/OWNERS~~ [tengqm]
Approvers can indicate their approval by writing /approve
in a comment
Approvers can cancel approval by writing /approve cancel
in a comment
@tengqm @dipesh-rawat Sorry for the delay. I addressed the last comment. Back to you!
/lgtm
LGTM label has been added.