On 4/15/26 8:31 AM, Ales Musil wrote: > 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]> >
Thanks for the review, Ales, applied to main and 26.03. Regards, Dumitru _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
