Skip to content

Commit

Permalink
KEP-127: Update to latest KEP template
Browse files Browse the repository at this point in the history
The KEP template was updated 2 weeks ago. Update to the latest template
and answer the new question.

Signed-off-by: Rodrigo Campos <[email protected]>
  • Loading branch information
rata committed Feb 7, 2023
1 parent fb04707 commit 734207a
Showing 1 changed file with 27 additions and 0 deletions.
27 changes: 27 additions & 0 deletions keps/NNNN-kep-template/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -732,6 +732,33 @@ Are there any tests that were run/should be run to understand performance charac
and validate the declared limits?
-->

###### Can enabling / using this feature result in resource exhaustion of some node resources (PIDs, sockets, inodes, etc.)?

<!--
Focus not just on happy cases, but primarily on more pathological cases
(e.g. probes taking a minute instead of milliseconds, failed pods consuming resources, etc.).
If any of the resources can be exhausted, how this is mitigated with the existing limits
(e.g. pods per node) or new limits added by this KEP?
Are there any tests that were run/should be run to understand performance characteristics better
and validate the declared limits?
-->

The kubelet is spliting the host UID/GID space for different pods, to use for
their user namespace mapping. The design allows for 65k pods per node, and the
resource is limited in the alpha phase to the min between maxPods per node
kubelet setting and 1024. This guarantees we are not inadvertly exhausting the
resource.

For container runtimes, they might use more disk space or inodes to chown the
rootfs. This is if they chose to support this feature without relying on new
Linux kernels (or supporting old kernels too), as new kernels allow idmap mounts
and no overhead (space nor inodes) is added with that.

For CRIO and containerd, we are working to incrementally support all variations
(idmap mounts, no overhead;overlyafs metacopy param, that gives us just inode
overhead; and a full rootfs chown, that has space overhead) and document them
appropiately.

### Troubleshooting

<!--
Expand Down

0 comments on commit 734207a

Please sign in to comment.