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

Netlink debug #15

Closed
wants to merge 3 commits into from
Closed

Conversation

yeazelm
Copy link
Contributor

@yeazelm yeazelm commented Jan 2, 2025

Recut of PR to core kit: bottlerocket-os/bottlerocket-core-kit#289

Issue number:

Closes # bottlerocket-os/bottlerocket#4313

Description of changes:
CRIU requires NETLINK_DEBUG for capturing socket info for checkpoint and restore. This adds the module to the kernel config for that use case.

The other PR was approved but was not merged before we moved the kernels into the kernel kit, this is moving that change to the new kit.

Testing done:
Built aws-k8s-1.31 and confirmed module loads.

bash-5.1# modprobe netlink_diag
bash-5.1# lsmod | grep netlink
netlink_diag           16384  0
nf_conntrack_netlink    57344  0
nfnetlink_acct         16384  3 xt_nfacct
nfnetlink              20480  7 nfnetlink_acct,nf_conntrack_netlink,nf_tables
nf_conntrack          184320  7 xt_conntrack,nf_nat,xt_state,xt_nat,nf_conntrack_netlink,xt_connmark,xt_MASQUERADE

Confirmed the same works on aws-k8s-1.24 (5.15 kernel) and aws-ecs-1 (5.10 kernel). I also checked aarch64 for 6.1 kernel.

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.

CRIU requires NETLINK_DEBUG for capturing socket info for checkpoint and
restore. This adds the module for that use case.

Signed-off-by: Matthew Yeazel <[email protected]>
CRIU requires NETLINK_DEBUG for capturing socket info for checkpoint and
restore. This adds the module for that use case.

Signed-off-by: Matthew Yeazel <[email protected]>
CRIU requires NETLINK_DEBUG for capturing socket info for checkpoint and
restore. This adds the module for that use case.

Signed-off-by: Matthew Yeazel <[email protected]>
@yeazelm
Copy link
Contributor Author

yeazelm commented Jan 27, 2025

#25 has this change upstream. We can merge now that it is present.

@yeazelm yeazelm closed this Jan 27, 2025
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.

3 participants