Skip to content

Conversation

@timbuchwaldt
Copy link

@timbuchwaldt timbuchwaldt commented Dec 1, 2025

Enable Netkit flag

Enable the netkit config flag, closes flatcar/Flatcar#1959

How to use

Load netkit module, enjoy Cilium performance improvements.

Testing done

[Describe the testing you have done before submitting this PR. Please include both the commands you issued as well as the output you got.]

  • Changelog entries added in the respective changelog/ directory (user-facing change, bug fix, security fix, update)
  • Inspected CI output for image differences: /boot and /usr size, packages, list files for any missing binaries, kernel modules, config files, kernel modules, etc.

@timbuchwaldt timbuchwaldt requested a review from a team as a code owner December 1, 2025 12:04
@timbuchwaldt timbuchwaldt force-pushed the main branch 3 times, most recently from fc75bc9 to 9c42188 Compare December 1, 2025 12:07
Closes flatcar/Flatcar#1959
Signed-off-by: Tim Buchwaldt <tim.buchwaldt@deepl.com>
@tormath1
Copy link
Contributor

tormath1 commented Dec 2, 2025

Images are available here: https://github.com/flatcar/scripts/actions/runs/19855010770?pr=3524

and netkit is enabled:

$ zgrep -i netkit /proc/config.gz
CONFIG_NETKIT=y

Two tests are failing but it's unrelated (because the PR comes from a fork).

@timbuchwaldt any chance to try the provided images?

@chewi
Copy link
Contributor

chewi commented Dec 2, 2025

Will this make our Cilium tests more reliable? 🤔

@tormath1
Copy link
Contributor

tormath1 commented Dec 2, 2025

Will this make our Cilium tests more reliable? 🤔

Not really, it seems it has to be explicitely opt-in. What we can do is to add a variant to our Cilium Kubernetes tests to test netkit, similar to this: https://github.com/flatcar/mantle/blob/c4e298d8328977f03869d2f73ddd4c61cc836a0d/kola/tests/kubeadm/kubeadm.go#L50-L54

@timbuchwaldt
Copy link
Author

Currently out sick from work, will have a look at testing once back

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[RFE] Enable CONFIG_NETKIT

3 participants