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

kernel: increase max_dgram_qlen sysctl value to 512 #3139

Conversation

markusboehme
Copy link
Member

Issue number: #3059

Description of changes: Increase the kernel's default value for the net.unix.max_dgram_qlen sysctl to 512. This is a change to the kernel rather than a plain sysctl setting since systemd-sysctl only applies settings to the host, while the changed default value in the kernel also applies to every new network namespace.

I will propose this upstream as well. In case of rejection, Bottlerocket can continue to carry this downstream without any expected maintenance effort.

Testing done:

With a plain Fedora pod (test) on a aws-k8s-1.23 x86_64 node running

...with this change:

$ kubectl exec test -- cat /proc/sys/net/unix/max_dgram_qlen
512

...without this change:

$ kubectl exec test -- cat /proc/sys/net/unix/max_dgram_qlen
10

Terms of contribution:

By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.

Increase the kernel's default value for the net.unix.max_dgram_qlen sysctl
to 512. This is a change to the kernel rather than a plain sysctl setting
since systemd-sysctl only applies settings to the host, while the changed
default value in the kernel also applies to every new network namespace.

Signed-off-by: Markus Boehme <[email protected]>
Increase the kernel's default value for the net.unix.max_dgram_qlen sysctl
to 512. This is a change to the kernel rather than a plain sysctl setting
since systemd-sysctl only applies settings to the host, while the changed
default value in the kernel also applies to every new network namespace.

Signed-off-by: Markus Boehme <[email protected]>
@etungsten etungsten merged commit d7575b2 into bottlerocket-os:develop May 23, 2023
@diranged
Copy link

@etungsten amazing, thank you!

etungsten added a commit that referenced this pull request May 24, 2023
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.

5 participants