Skip to content

Conversation

nbusseneau
Copy link
Member

@nbusseneau nbusseneau commented May 15, 2025

julianwiedmann and others added 6 commits May 15, 2025 18:29
[ upstream commit d70929b ]

The BPF_HAVE_NETNS_COOKIE macro doesn't exist in-tree, use the correct
spelling.

Looks like the PR that introduced this test conflicted with the renaming in
17a652b ("probes: remove 'BPF_' prefix from features macros").

Signed-off-by: Julian Wiedmann <jwi@isovalent.com>
Signed-off-by: Nicolas Busseneau <nicolas@isovalent.com>
[ upstream commit 1276096 ]

Add a section to talk about the native routing masquerading in the cloud
environment based on discussion
#39156 (comment)

Signed-off-by: Liyi Huang <liyi.huang@isovalent.com>
Signed-off-by: Nicolas Busseneau <nicolas@isovalent.com>
[ upstream commit 3fb8618 ]

[ backporter's notes: some conflicts due to older test setup. ]

We can no longer treat single-selector policies as wildcarding L3, as we
no longer have bpf datapath always performing policy enforcement before
cilium-envoy.

Signed-off-by: Jarno Rajahalme <jarno@isovalent.com>
Signed-off-by: Nicolas Busseneau <nicolas@isovalent.com>
[ upstream commit d9c3afc ]

It's not required to validate traffic that is re-injected by the proxy
i.e. the original traffic was originally redirected to proxy, and then
came back.

With this change, there is more flexibility on setting the upstream
connection src IP from proxy.

Signed-off-by: Tam Mach <tam.mach@cilium.io>
Signed-off-by: Nicolas Busseneau <nicolas@isovalent.com>
[ upstream commit 9394900 ]

During some other work I discovered that the whole active connection
tracking feature does not compile when enabled. Adding to the complexity
tests to add some compile coverage as regression test. Will fix the
actual issue in a subsequent commit.

Signed-off-by: Dylan Reimerink <dylan.reimerink@isovalent.com>
Signed-off-by: Nicolas Busseneau <nicolas@isovalent.com>
[ upstream commit 7c27078 ]

When both IPv6 and active connection tracking are enabled, we are unable
to compile, resulting in the following error:

```
./lib/lb.h:1071:21: error: use of undeclared identifier 'ct_state'
```

This is because in `lb6_local` the name of the variable is `state` not
`ct_state`. This issue seems to have been here since the introduction
of the feature and was never caught due to a lack of testing.

Signed-off-by: Dylan Reimerink <dylan.reimerink@isovalent.com>
Signed-off-by: Nicolas Busseneau <nicolas@isovalent.com>
@nbusseneau nbusseneau added kind/backports This PR provides functionality previously merged into master. backport/1.16 This PR represents a backport for Cilium 1.16.x of a PR that was merged to main. labels May 15, 2025
@nbusseneau
Copy link
Member Author

/test

@nbusseneau nbusseneau marked this pull request as ready for review May 15, 2025 20:53
@nbusseneau nbusseneau requested a review from a team as a code owner May 15, 2025 20:53
Copy link
Member

@sayboras sayboras left a comment

Choose a reason for hiding this comment

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

Thanks and looks good for my commit

@nbusseneau nbusseneau added this pull request to the merge queue May 19, 2025
Merged via the queue into v1.16 with commit f1c44a0 May 19, 2025
292 of 293 checks passed
@nbusseneau nbusseneau deleted the pr/v1.16-backport-2025-05-15-06-29 branch May 19, 2025 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/1.16 This PR represents a backport for Cilium 1.16.x of a PR that was merged to main. kind/backports This PR provides functionality previously merged into master.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants