website icon indicating copy to clipboard operation
website copied to clipboard

clarify CPU and memory limit enforcement differences

Open haircommander opened this issue 1 year ago • 6 comments

Clairify the container runtime and kubelet do not enforece memory limits, the kernel does. Also clarify that workloads won't always be OOM killed, and more clearly spell out the differences between memory and CPU limits

haircommander avatar May 06 '24 14:05 haircommander

Pull request preview available for checking

Built without sensitive environment variables

Name Link
Latest commit 0666011822b3ebbec179d6b3a2e6832977daa3bc
Latest deploy log https://app.netlify.com/sites/kubernetes-io-main-staging/deploys/6723af7f44f30000088a3ac7
Deploy Preview https://deploy-preview-46222--kubernetes-io-main-staging.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] avatar May 06 '24 14:05 netlify[bot]

@kubernetes/sig-node-pr-reviews FYI

sftim avatar May 06 '24 23:05 sftim

@haircommander Whenever you get a chance, please review the feedback from the reviewers and respond to them accordingly. Thanks!

dipesh-rawat avatar May 25 '24 00:05 dipesh-rawat

The Kubernetes project currently lacks enough contributors to adequately respond to all PRs.

This bot triages PRs according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the PR is closed

You can:

  • Mark this PR as fresh with /remove-lifecycle stale
  • Close this PR with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle stale

k8s-triage-robot avatar Aug 23 '24 01:08 k8s-triage-robot

The Kubernetes project currently lacks enough active contributors to adequately respond to all PRs.

This bot triages PRs according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the PR is closed

You can:

  • Mark this PR as fresh with /remove-lifecycle rotten
  • Close this PR with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle rotten

k8s-triage-robot avatar Sep 22 '24 02:09 k8s-triage-robot

@haircommander : Gentle reminder!

divya-mohan0209 avatar Oct 31 '24 11:10 divya-mohan0209

I finally dug out some time to respond! I have updated and clarified some sections

haircommander avatar Oct 31 '24 16:10 haircommander

The clarification is lovely, @haircommander, I have a better understanding now. I'm looking forward to feedback from others.

network-charles avatar Oct 31 '24 18:10 network-charles

/lgtm

tengqm avatar Nov 01 '24 00:11 tengqm

LGTM label has been added.

Git tree hash: 357a049c42a8211dbf1a1c792b03fce44eb5245a

k8s-ci-robot avatar Nov 01 '24 00:11 k8s-ci-robot

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: sftim

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 /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

k8s-ci-robot avatar Nov 25 '24 00:11 k8s-ci-robot