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 net.ipv4.ip_unprivileged_port_start sysctl #298

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

pacoxu
Copy link
Contributor

@pacoxu pacoxu commented Aug 6, 2024

kubernetes/kubernetes#103326 marked it as safe sysctl since Kubernetes v1.22.

Kernel 4.11 add this: torvalds/linux@4548b68 which is per namespaced.

xref coredns/coredns#6716 and kubernetes/kubernetes#125226.

Signed-off-by: Paco Xu <paco.xu@daocloud.io>
Signed-off-by: Paco Xu <paco.xu@daocloud.io>
@pacoxu
Copy link
Contributor Author

pacoxu commented Aug 6, 2024

CI failure is golang install failure.
I sent a PR #299 to fix the CI seperately.

@pacoxu
Copy link
Contributor Author

pacoxu commented Aug 6, 2024

/assign @chrisohaver

@pacoxu
Copy link
Contributor Author

pacoxu commented Aug 6, 2024

BTW, this needs kernel 4.11+

Or coredns pod will fail with below error:

Warning FailedCreatePodSandBox 2s (x13 over 43s) kubelet Failed to create pod sandbox: rpc error: code = Unknown desc = failed to create containerd task: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: error during container init: open /proc/sys/net/ipv4/ip_unprivileged_port_start: no such file or directory: unknown

For more context, see kubernetes/kubernetes#105309 (comment).

@pacoxu
Copy link
Contributor Author

pacoxu commented Aug 6, 2024

/hold
for kernel version 4.11 requirement

@Tej-Singh-Rana
Copy link

Thanks, Pacoxu. 👍

@pacoxu pacoxu marked this pull request as draft August 7, 2024 01:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants