-
Notifications
You must be signed in to change notification settings - Fork 3.4k
Closed as not planned
Labels
area/CIContinuous Integration testing issue or flakeContinuous Integration testing issue or flakearea/datapathImpacts bpf/ or low-level forwarding details, including map management and monitor messages.Impacts bpf/ or low-level forwarding details, including map management and monitor messages.feature/snatRelates to SNAT or Masquerading of trafficRelates to SNAT or Masquerading of trafficneeds/e2e-testThis issue is not covered by existing CI tests, but should be.This issue is not covered by existing CI tests, but should be.staleThe stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale.The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale.
Description
Cilium v1.16 had a regression where in-cluster traffic was incorrectly SNATed and could not be matched to the source's security identity: #34615
Specifically, this affected IPAM modes "eni", "azure" and "alibabacloud" when ipv4-native-routing-CIDR
option was not explicitly set.
The issue has been addressed in both v1.16 and main branches with, respectively, #35611 and #35624.
An e2e test should be added to verify that in-cluster traffic is not SNATed, following the repro depicted in #34615 itself (that is, apply a CNP to allow ingress traffic from pods on remote nodes, but not traffic from the remote nodes themselves).
joestringer
Metadata
Metadata
Assignees
Labels
area/CIContinuous Integration testing issue or flakeContinuous Integration testing issue or flakearea/datapathImpacts bpf/ or low-level forwarding details, including map management and monitor messages.Impacts bpf/ or low-level forwarding details, including map management and monitor messages.feature/snatRelates to SNAT or Masquerading of trafficRelates to SNAT or Masquerading of trafficneeds/e2e-testThis issue is not covered by existing CI tests, but should be.This issue is not covered by existing CI tests, but should be.staleThe stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale.The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale.