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

Add sysctl proposal #700

Merged
merged 2 commits into from
Jun 12, 2017
Merged

Add sysctl proposal #700

merged 2 commits into from
Jun 12, 2017

Conversation

sttts
Copy link
Contributor

@sttts sttts commented Jun 9, 2017

Follow-up of kubernetes/kubernetes#26057. The proposal is implemented (with the limited set of safe sysctls). So this PR is only to preserve the proposal text, as the basis for further work in this direction.

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Jun 9, 2017
@sttts sttts force-pushed the sttts-sysctl-proposal branch from b0a151a to b6999f4 Compare June 9, 2017 07:14
@sttts
Copy link
Contributor Author

sttts commented Jun 9, 2017

@derekwaynecarr fyi. I guess we could open up more sysctls now as ipc accounting is possible.

@vishh
Copy link
Contributor

vishh commented Jun 9, 2017

LGTM

@vishh
Copy link
Contributor

vishh commented Jun 9, 2017

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jun 9, 2017

- pre-requisites for `kernel.sem`, `kernel.msg*`, `fs.mqueue.*` on the node-level whitelist
+ [x] pod cgroups active by default (compare [Pod Resource Management](pod-resource-management.md#implementation-status))
+ [ ] kmem accounting active by default
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vishh @derekwaynecarr any status update on this?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not even in our roadmap for the next year.

@sttts sttts merged commit 8dbec90 into kubernetes:master Jun 12, 2017
@sttts sttts deleted the sttts-sysctl-proposal branch June 12, 2017 14:34
MadhavJivrajani pushed a commit to MadhavJivrajani/community that referenced this pull request Nov 30, 2021
danehans pushed a commit to danehans/community that referenced this pull request Jul 18, 2023
* add shuaijinchao as member

* fix ci members unsorted
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants