On 9/1/25 1:22 PM, Dumitru Ceara wrote:
> The Southbound Static_Mac_Binding table is write-only from ovn-northd
> perspective.  Omit alerting for any changes on this table (as we do for
> all other write-only tables) in ovn-northd.
> 
> Fixes: b22684d4e37c ("Add a northbound interface to program MAC_Binding 
> table")
> Signed-off-by: Dumitru Ceara <dce...@redhat.com>
> ---
>  northd/ovn-northd.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c
> index 62da4a3aad..29152cdd74 100644
> --- a/northd/ovn-northd.c
> +++ b/northd/ovn-northd.c
> @@ -962,6 +962,10 @@ main(int argc, char *argv[])
>          ovsdb_idl_omit_alert(ovnsb_idl_loop.idl,
>                               &sbrec_advertised_route_columns[i]);
>      }
> +    for (size_t i = 0; i < SBREC_STATIC_MAC_BINDING_N_COLUMNS; i++) {
> +        ovsdb_idl_omit_alert(ovnsb_idl_loop.idl,
> +                             &sbrec_static_mac_binding_columns[i]);
> +    }
>  
>      unixctl_command_register("sb-connection-status", "", 0, 0,
>                               ovn_conn_show, ovnsb_idl_loop.idl);

FWIW, looks like this test breaks the static mac binding garbage collection
test in GitHub CI.

Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to