Skip to content

Commit 2c7ba78

Browse files
pliurhjcaamano
authored andcommitted
Import BGP routes to UDN when EgressIP is advertised
BGP routes will be imported to the UDN VRF and GR, when EgressIP for the UDN is advertised to the default VRF. This ensures that the egress traffic can find the right egress route. Signed-off-by: Peng Liu <[email protected]>
1 parent bb78ce8 commit 2c7ba78

File tree

2 files changed

+2
-6
lines changed

2 files changed

+2
-6
lines changed

go-controller/pkg/clustermanager/routeadvertisements/controller.go

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -714,11 +714,6 @@ func (c *Controller) generateFRRConfiguration(
714714
continue
715715
}
716716

717-
// we won't do imports if the pod network is not advertised
718-
if !advertisements.Has(ratypes.PodNetwork) {
719-
continue
720-
}
721-
722717
// before handling imports, lets normalize the VRF for the default
723718
// network: when doing imports, the default VRF is is referred to as
724719
// "default" instead of ""

go-controller/pkg/clustermanager/routeadvertisements/controller_test.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -551,7 +551,8 @@ func TestController_reconcile(t *testing.T) {
551551
Routers: []*testRouter{
552552
{ASN: 1, VRF: "red", Prefixes: []string{"1.0.1.3/32", "172.100.0.16/32"}, Neighbors: []*testNeighbor{
553553
{ASN: 1, Address: "1.0.0.100", Advertise: []string{"1.0.1.3/32", "172.100.0.16/32"}},
554-
}},
554+
}, Imports: []string{"blue"}},
555+
{ASN: 1, VRF: "blue", Imports: []string{"red"}},
555556
}},
556557
},
557558
expectNADAnnotations: map[string]map[string]string{"blue": {types.OvnRouteAdvertisementsKey: "[\"ra\"]"}},

0 commit comments

Comments
 (0)