Skip to content

Conversation

borkmann
Copy link
Member

(see commit desc)

@borkmann borkmann added area/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages. release-note/minor This PR changes functionality that users may find relevant to operating Cilium. labels Nov 22, 2024
@borkmann borkmann changed the title cilium: Add option for lb src ranges to act as deny list cilium: Add option for lb src ranges to act as deny cidr list Nov 22, 2024
@borkmann borkmann force-pushed the pr/lbsrcranges branch 2 times, most recently from b0a195d to e8cf5a1 Compare November 25, 2024 15:45
Extend BPF side to add a service flag for turning the
loadBalancerSourceRanges into a deny CIDR list. The flag
used is always on the master entry, actual non-master entries
use the flag to indicate that a service backend is quarantined.
Both usage is guaranteed to not overlap.

Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
@borkmann borkmann force-pushed the pr/lbsrcranges branch 2 times, most recently from 39e3eac to c4735f1 Compare November 26, 2024 10:14
@borkmann borkmann marked this pull request as ready for review November 26, 2024 10:27
@borkmann borkmann requested review from a team as code owners November 26, 2024 10:27
@borkmann borkmann requested review from youngnick and a user November 26, 2024 10:27
Add agent-side handling of the source ranges policy annotation in order
to plumb the flag through into the BPF service map.

Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Extend the KPR guide to document the loadBalancerSourceRanges extension
along with usage examples.

Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
@borkmann
Copy link
Member Author

/test

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

docs good

@borkmann borkmann merged commit 0127b24 into main Nov 26, 2024
282 of 283 checks passed
@borkmann borkmann deleted the pr/lbsrcranges branch November 26, 2024 13:01
@julianwiedmann julianwiedmann added the area/loadbalancing Impacts load-balancing and Kubernetes service implementations label Nov 26, 2024
joamaki added a commit to joamaki/cilium that referenced this pull request May 14, 2025
This ports cilium#36120 to the new control-plane.

Signed-off-by: Jussi Maki <jussi@isovalent.com>
joamaki added a commit to joamaki/cilium that referenced this pull request May 14, 2025
This ports cilium#36120 to the new control-plane.

Signed-off-by: Jussi Maki <jussi@isovalent.com>
joamaki added a commit to joamaki/cilium that referenced this pull request May 19, 2025
This ports cilium#36120 to the new control-plane.

Signed-off-by: Jussi Maki <jussi@isovalent.com>
github-merge-queue bot pushed a commit that referenced this pull request May 19, 2025
This ports #36120 to the new control-plane.

Signed-off-by: Jussi Maki <jussi@isovalent.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages. area/loadbalancing Impacts load-balancing and Kubernetes service implementations release-note/minor This PR changes functionality that users may find relevant to operating Cilium.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants