Message ID | ad13e8d7f3b55439dbf3af15af3d5f0552cdad73.1705487435.git.lorenzo.bianconi@redhat.com |
---|---|
State | Changes Requested |
Headers | show |
Series | Introduce BFD support for ECMP route policy | expand |
Context | Check | Description |
---|---|---|
ovsrobot/apply-robot | success | apply and check: success |
ovsrobot/github-robot-_Build_and_Test | success | github build: passed |
ovsrobot/github-robot-_ovn-kubernetes | success | github build: passed |
Thanks Lorenzo! Acked-by: Mark Michelson <mmichels@redhat.com> On 1/17/24 06:11, Lorenzo Bianconi wrote: > In order to properly check if we already have a running BFD session for > the next hop used by the brand new route, load logical port column of > the nb BFD table in nbctl_pre_lr_route_add routine. > > Fixes: db6d30783bec ("ovn-nbctl: Don't replicate entire database unnecessarily.") > Signed-off-by: Lorenzo Bianconi <lorenzo.bianconi@redhat.com> > --- > utilities/ovn-nbctl.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/utilities/ovn-nbctl.c b/utilities/ovn-nbctl.c > index 526369b68..0586eccdb 100644 > --- a/utilities/ovn-nbctl.c > +++ b/utilities/ovn-nbctl.c > @@ -4394,6 +4394,7 @@ nbctl_pre_lr_route_add(struct ctl_context *ctx) > ovsdb_idl_add_column(ctx->idl, &nbrec_logical_router_port_col_name); > > ovsdb_idl_add_column(ctx->idl, &nbrec_bfd_col_dst_ip); > + ovsdb_idl_add_column(ctx->idl, &nbrec_bfd_col_logical_port); > > ovsdb_idl_add_column(ctx->idl, > &nbrec_logical_router_static_route_col_ip_prefix);
diff --git a/utilities/ovn-nbctl.c b/utilities/ovn-nbctl.c index 526369b68..0586eccdb 100644 --- a/utilities/ovn-nbctl.c +++ b/utilities/ovn-nbctl.c @@ -4394,6 +4394,7 @@ nbctl_pre_lr_route_add(struct ctl_context *ctx) ovsdb_idl_add_column(ctx->idl, &nbrec_logical_router_port_col_name); ovsdb_idl_add_column(ctx->idl, &nbrec_bfd_col_dst_ip); + ovsdb_idl_add_column(ctx->idl, &nbrec_bfd_col_logical_port); ovsdb_idl_add_column(ctx->idl, &nbrec_logical_router_static_route_col_ip_prefix);
In order to properly check if we already have a running BFD session for the next hop used by the brand new route, load logical port column of the nb BFD table in nbctl_pre_lr_route_add routine. Fixes: db6d30783bec ("ovn-nbctl: Don't replicate entire database unnecessarily.") Signed-off-by: Lorenzo Bianconi <lorenzo.bianconi@redhat.com> --- utilities/ovn-nbctl.c | 1 + 1 file changed, 1 insertion(+)