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

Hugepages PRs Tracker #84526

Closed
bg-chun opened this issue Oct 29, 2019 · 4 comments
Closed

Hugepages PRs Tracker #84526

bg-chun opened this issue Oct 29, 2019 · 4 comments
Assignees
Labels
kind/feature Categorizes issue or PR as related to a new feature. sig/node Categorizes an issue or PR as relevant to SIG Node.

Comments

@bg-chun
Copy link
Member

bg-chun commented Oct 29, 2019

Note : The issue in enhancement repo will be used for PRs Tracker kubernetes/enhancements#1539


This issue is Hugepages PRs Traker.
This google doc contains more information for these PRs.

Hugepages KEP have been updated for serveral enhancements.

  • Support container isolation of hugepages / KEP Update1(merged)
  • Support multi size hugepages at host level / KEP Update2(merged)
  • Support multi size hugepages at container level / KEP Update2(merged)
  • Support hugepage reservation for system-level service / part of original KEP

Now, We have some pending PRs for hugepages.
Some of them reviewed, but most of them need review, they need attention.

Pending PRs

  • Note: * means the PR is the top of dependency.
PR Description Status Dep Owner
#83614* Update CRI to support hugepages merged   @bg-chun
#84154 Support for setting hugepages limit during container creation merged #83614 @ohsewon
#84701 Update Dockershim WIP #84154 @admanV
M40160 Add hugepages field to resource(moby) change requrested #83614 @bg-chun
C1332 Update Container runtimes(containerd) got LGTM #84154 @bg-chun
C2940 Update Container runtimes(cri-o) Merged #84154 @bg-chun
#87118 e2e_node test for container isolation of hugepage need review #84154 @ohsewon
#82820* Support for pre-allocated hugepages with 2+ sizes(for host) Merged   @odinuge
#84051 Support for multiple sizes huge pages(for pod) need sig review #82820 @bart0sh
#80831 Add support for removing unsupported huge page sizes got lgtm   @odinuge
#83541 Support for reserving hugepages for system and kubelet need 2rd review   @odinuge
#80605 Add huge page usage stats to kubectl describe node need CLI review   @odinuge
#81774 Bugfix: Kubelet doesn’t update /sys/fs/cgroup/hugetlb/kubepods/hugetlb.2MB.limit_in_bytes upon Node Status Update got lgtm   @rojkov
@bg-chun bg-chun added the kind/feature Categorizes issue or PR as related to a new feature. label Oct 29, 2019
@k8s-ci-robot k8s-ci-robot added the needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. label Oct 29, 2019
@bg-chun
Copy link
Member Author

bg-chun commented Oct 29, 2019

/sig node

@k8s-ci-robot k8s-ci-robot added sig/node Categorizes an issue or PR as relevant to SIG Node. and removed needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. labels Oct 29, 2019
@bg-chun
Copy link
Member Author

bg-chun commented Oct 29, 2019

/assign dchen1107

@odinuge
Copy link
Member

odinuge commented Nov 6, 2019

I think #80831 should be added to the list too.

@bg-chun
Copy link
Member Author

bg-chun commented Nov 6, 2019

@odinuge I added it :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature. sig/node Categorizes an issue or PR as relevant to SIG Node.
Projects
None yet
Development

No branches or pull requests

5 participants