On Tue, Apr 14, 2026 at 3:41 PM Dumitru Ceara <[email protected]> wrote:
> In northd_handle_ls_changes(), init_mcast_info_for_datapath() is called
> after ovn_datapath_create(), but ovn_datapath_create() already calls it
> internally. Remove the redundant call.
>
> Fixes: 06e2c1bf0ce7 ("northd: I-P for logical switch creation/deletion in
> en_northd.")
> Assisted-by: Claude, with model: claude-opus-4-6
> Signed-off-by: Dumitru Ceara <[email protected]>
> ---
> northd/northd.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/northd/northd.c b/northd/northd.c
> index 6a8d0c311a..bc817073e2 100644
> --- a/northd/northd.c
> +++ b/northd/northd.c
> @@ -5138,7 +5138,6 @@ northd_handle_ls_changes(struct ovsdb_idl_txn
> *ovnsb_idl_txn,
> ods_assign_array_index(&nd->ls_datapaths, od);
> init_ipam_info_for_datapath(od);
> init_ls_other_config(od);
> - init_mcast_info_for_datapath(od);
>
> /* Create SB:IP_Multicast for the logical switch. */
> const struct sbrec_ip_multicast *ip_mcast =
> --
> 2.53.0
>
>
Looks good to me thanks.
Acked-by: Ales Musil <[email protected]>
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
