Add route summarization to bgp advertistments services#36791
Add route summarization to bgp advertistments services#36791romanspb80 wants to merge 3 commits intocilium:mainfrom
Conversation
ef13b67 to
b8fbbc7
Compare
b8fbbc7 to
9ab1f5d
Compare
New summarization option under the CiliumLoadBalancerIPPool to summarize address apiVersion: "cilium.io/v2alpha1" kind: CiliumLoadBalancerIPPool metadata: name: "lab-pool" spec: summarize: true <------- new option to summarize any service to the block address when advertised blocks: - cidr: "10.100.11.0/24" Fixes: cilium#31116 Signed-off-by: Roman Ptitcyn <romanspb@yahoo.com>
New summarization option under the CiliumLoadBalancerIPPool to summarize address Fixes: cilium#31116 Signed-off-by: Roman Ptitcyn <romanspb@yahoo.com>
New summarization option under the CiliumLoadBalancerIPPool to summarize address Fixes: cilium#31116 Signed-off-by: Roman Ptitcyn <romanspb@yahoo.com>
9ab1f5d to
685f5be
Compare
|
Hi @romanspb80 Thanks for your contribution, I see that feature request issue regarding route-aggregation was raised recently. So, implementation is very timely. I have put my thoughts in this comment, I think we need to come up with approach which addresses those concerns. I suggest putting this PR in draft for time being till we conclude on scope and design. You can add additional context in that issue as well. |
Hi @harsimran-pabla . Thanks |
|
Hi @romanspb80 please let me know if you plan on using same PR for the new implementation. I'd suggest closing this one and opening new PR with agreed approach. You can open a draft PR and tag sig-bgp for initial review before pulling in other reviewers. |
Hi @harsimran-pabla . I've rised PR for BGP Control Plane version2 in accordance we discussed that case. I haven't closed this PR yet, because it might make sense to implement aggregation for the BGP Control Plane version1 as well, while it's still relevant. |
|
From the community meeting: It looks like #37275 is superseding this PR, so there is no need for this PR. Closing. |
New summarization option under the CiliumLoadBalancerIPPool to summarize address
apiVersion: "cilium.io/v2alpha1"
kind: CiliumLoadBalancerIPPool
metadata:
name: "lab-pool"
spec:
summarize: true <------- new option to summarize any service to the block address when advertised
blocks:
cidr: "10.100.11.0/24"
Fixes: #31116