-
Notifications
You must be signed in to change notification settings - Fork 14.4k
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
Documentation for in-place pod resize feature #39845
Conversation
bb7d86c
to
b893d8b
Compare
/assign @sftim |
content/en/docs/tasks/configure-pod-container/resize-container-resources.md
Outdated
Show resolved
Hide resolved
content/en/docs/tasks/configure-pod-container/resize-container-resources.md
Outdated
Show resolved
Hide resolved
content/en/docs/tasks/configure-pod-container/resize-container-resources.md
Outdated
Show resolved
Hide resolved
content/en/docs/tasks/configure-pod-container/resize-container-resources.md
Outdated
Show resolved
Hide resolved
content/en/docs/tasks/configure-pod-container/resize-container-resources.md
Outdated
Show resolved
Hide resolved
content/en/docs/tasks/configure-pod-container/resize-container-resources.md
Outdated
Show resolved
Hide resolved
content/en/docs/tasks/configure-pod-container/resize-container-resources.md
Outdated
Show resolved
Hide resolved
content/en/docs/tasks/configure-pod-container/resize-container-resources.md
Outdated
Show resolved
Hide resolved
content/en/docs/tasks/configure-pod-container/resize-container-resources.md
Outdated
Show resolved
Hide resolved
content/en/docs/tasks/configure-pod-container/resize-container-resources.md
Outdated
Show resolved
Hide resolved
@tengqm Thanks for the careful review, much appreciated! I fixed them (and hopefully didn't miss any of the items). PTAL. |
content/en/docs/tasks/configure-pod-container/resize-container-resources.md
Outdated
Show resolved
Hide resolved
/label tide/merge-method-squash |
LGTM label has been added. Git tree hash: 90754a44e41d00cec39a6c8ed1f0629e8c5b5bd9
|
@tengqm I noticed a few missing things. Please review latest commit. thanks |
a1bc39f
to
e5ebc5a
Compare
content/en/docs/tasks/configure-pod-container/resize-container-resources.md
Outdated
Show resolved
Hide resolved
content/en/docs/tasks/configure-pod-container/resize-container-resources.md
Outdated
Show resolved
Hide resolved
content/en/docs/tasks/configure-pod-container/resize-container-resources.md
Outdated
Show resolved
Hide resolved
content/en/docs/tasks/configure-pod-container/resize-container-resources.md
Outdated
Show resolved
Hide resolved
content/en/docs/tasks/configure-pod-container/resize-container-resources.md
Outdated
Show resolved
Hide resolved
content/en/docs/tasks/configure-pod-container/resize-container-resources.md
Outdated
Show resolved
Hide resolved
content/en/docs/tasks/configure-pod-container/resize-container-resources.md
Outdated
Show resolved
Hide resolved
content/en/docs/tasks/configure-pod-container/resize-container-resources.md
Outdated
Show resolved
Hide resolved
content/en/docs/tasks/configure-pod-container/resize-container-resources.md
Outdated
Show resolved
Hide resolved
content/en/docs/tasks/configure-pod-container/resize-container-resources.md
Outdated
Show resolved
Hide resolved
content/en/docs/tasks/configure-pod-container/resize-container-resources.md
Outdated
Show resolved
Hide resolved
@vinaykul Please try keep this PR at a single commit, i.e., please squash your commits into one. |
2235f3c
to
d71d2e5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks. With a few tweaks this should be OK to merge.
content/en/docs/tasks/configure-pod-container/resize-container-resources.md
Show resolved
Hide resolved
content/en/docs/tasks/configure-pod-container/resize-container-resources.md
Show resolved
Hide resolved
content/en/docs/tasks/configure-pod-container/resize-container-resources.md
Outdated
Show resolved
Hide resolved
content/en/docs/tasks/configure-pod-container/resize-container-resources.md
Outdated
Show resolved
Hide resolved
The following changes enable in-place resize of container resources: | ||
- Container's resource **requests** and **limits** are now **mutable** for CPU | ||
and memory resources. | ||
- A new field named `allocatedResources` has been added to `containerStatuses` | ||
in the Pod's status to reflect the resources allocated to the pod's containers. | ||
- A new field named `resources` has been added to the Container's status. This | ||
field reflects the actual resource requests and limits that are configured on | ||
the running containers as reported by the container runtime. | ||
- A new field named `resize` has been added to the Pod's status to show the | ||
status of the last requested pending resize. It can have the following values: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Describe the current behavior, not the diff.
content/en/docs/tasks/configure-pod-container/resize-container-resources.md
Show resolved
Hide resolved
content/en/docs/tasks/configure-pod-container/resize-container-resources.md
Outdated
Show resolved
Hide resolved
|
||
Consider the following manifest for a Pod that has one Container. | ||
|
||
{{< codenew file="pods/qos/qos-pod.yaml" >}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider making a new separate example manifest.
content/en/docs/tasks/configure-pod-container/resize-container-resources.md
Show resolved
Hide resolved
content/en/docs/tasks/configure-pod-container/resize-container-resources.md
Show resolved
Hide resolved
- Container's resource **requests** and **limits** are now **mutable** for CPU | ||
and memory resources. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not a strong opinion, but I'd like to format bulletin items by having the following lines indented. For example:
- Container's resource **requests** and **limits** are now **mutable** for CPU | |
and memory resources. | |
- Container's resource **requests** and **limits** are now **mutable** for CPU | |
and memory resources. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
/approve |
[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
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Fix issues from review feedback Update content/en/docs/tasks/configure-pod-container/resize-container-resources.md Co-authored-by: Qiming Teng <tengqm@outlook.com> Better wording and formatting of overview section Add descriptions about allocatedResources, resources, and resize fields Update content/en/docs/tasks/configure-pod-container/resize-container-resources.md Co-authored-by: Qiming Teng <tengqm@outlook.com> Update content/en/docs/tasks/configure-pod-container/resize-container-resources.md Co-authored-by: Qiming Teng <tengqm@outlook.com> Apply suggestions from code review Co-authored-by: Qiming Teng <tengqm@outlook.com>
Co-authored-by: Tim Bannister <tim@scalefactory.com>
@sftim I fixed the issues you found. PTAL, thx |
@sftim Please take a look, I believe I have addressed all issues you found. |
/lgtm |
LGTM label has been added. Git tree hash: 5b8e5b235780d8e07749df16f0d47cea7b0b497a
|
* Documentation for in-place pod resize feature Fix issues from review feedback Update content/en/docs/tasks/configure-pod-container/resize-container-resources.md Co-authored-by: Qiming Teng <tengqm@outlook.com> Better wording and formatting of overview section Add descriptions about allocatedResources, resources, and resize fields Update content/en/docs/tasks/configure-pod-container/resize-container-resources.md Co-authored-by: Qiming Teng <tengqm@outlook.com> Update content/en/docs/tasks/configure-pod-container/resize-container-resources.md Co-authored-by: Qiming Teng <tengqm@outlook.com> Apply suggestions from code review Co-authored-by: Qiming Teng <tengqm@outlook.com> * Simplify the changes by using bullet points * Apply suggestions from code review Co-authored-by: Tim Bannister <tim@scalefactory.com> --------- Co-authored-by: Qiming Teng <tengqm@outlook.com> Co-authored-by: Tim Bannister <tim@scalefactory.com>
Documentation for in-place pod resize feature. Ref: kubernetes/kubernetes#102884