Skip to content

datapath/loader: Add netkit to BPF load tests#44225

Merged
julianwiedmann merged 1 commit intocilium:mainfrom
ajmmm:pr/netkit-complexity-testing
Feb 10, 2026
Merged

datapath/loader: Add netkit to BPF load tests#44225
julianwiedmann merged 1 commit intocilium:mainfrom
ajmmm:pr/netkit-complexity-testing

Conversation

@ajmmm
Copy link
Copy Markdown
Member

@ajmmm ajmmm commented Feb 6, 2026 •
edited
Loading

Commit 8544737 ("bpf: Workaround for netkit + L7 policy redirect failure") introduced the enable_netkit load-time config variable.

This commit introduces that variable to BPF loader permutation testing for bpf_lxc.

@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Feb 6, 2026
@ajmmm
Copy link
Copy Markdown
Member Author

ajmmm commented Feb 6, 2026

/test

@ajmmm
Copy link
Copy Markdown
Member Author

ajmmm commented Feb 6, 2026 •
edited
Loading

Ginkgo failure 1 is a flake: #13071

Ginkgo failure 2 is a flake: #44082

@ajmmm
Copy link
Copy Markdown
Member Author

ajmmm commented Feb 6, 2026

/ci-verifier

@ajmmm
datapath/loader: Add netkit to BPF load tests
a742fbc 
Commit 8544737 ("bpf: Workaround for netkit + L7 policy redirect failure")
introduced the enable_netkit load-time config variable.

This commit introduces that variable to BPF loader permutation testing
for bpf_lxc.

Signed-off-by: Alasdair McWilliam <alasdair.mcwilliam@isovalent.com>
@ajmmm ajmmm force-pushed the pr/netkit-complexity-testing branch from eec0142 to a742fbc Compare February 6, 2026 15:26
@ajmmm
Copy link
Copy Markdown
Member Author

ajmmm commented Feb 6, 2026

/test

@ajmmm ajmmm marked this pull request as ready for review February 6, 2026 16:25
@ajmmm ajmmm requested a review from a team as a code owner February 6, 2026 16:25
@ajmmm ajmmm requested a review from tommyp1ckles February 6, 2026 16:25
@julianwiedmann julianwiedmann added area/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages. release-note/ci This PR makes changes to the CI. feature/netkit labels Feb 10, 2026
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Feb 10, 2026
julianwiedmann
julianwiedmann approved these changes Feb 10, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@julianwiedmann julianwiedmann left a comment

Choose a reason for hiding this comment

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

thank you!

@julianwiedmann julianwiedmann added this pull request to the merge queue Feb 10, 2026
Merged via the queue into cilium:main with commit bd77681 Feb 10, 2026
80 checks passed
@ajmmm ajmmm deleted the pr/netkit-complexity-testing branch February 25, 2026 11:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@julianwiedmann julianwiedmann julianwiedmann approved these changes

@tommyp1ckles tommyp1ckles Awaiting requested review from tommyp1ckles tommyp1ckles is a code owner automatically assigned from cilium/sig-datapath

Assignees

No one assigned

Labels

area/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages. feature/netkit release-note/ci This PR makes changes to the CI.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ajmmm @julianwiedmann @msune