On 16 Nov 2023, at 12:24, David Marchand wrote:
> On Thu, Nov 16, 2023 at 10:38 AM Eelco Chaudron <echau...@redhat.com> wrote: >>> diff --git a/tests/mcast-snooping.at b/tests/mcast-snooping.at >>> index b5474cf392..1ce31168e8 100644 >>> --- a/tests/mcast-snooping.at >>> +++ b/tests/mcast-snooping.at >>> @@ -207,6 +207,24 @@ Megaflow: >>> recirc_id=0,eth,udp,in_port=3,dl_src=aa:55:aa:55:00:ff,dl_dst=01:00:5e >>> Datapath actions: 1,2 >>> ]) >>> >>> +AT_CHECK([ovs-vsctl set interface p2 ofport_request=4]) >> >> Can we add a comment here (and below) to indicate why we do this? Just to >> understand what we test here. > > Wdyt of: > +# Change p2 ofport to force a ofbundle change and check that the mdb contains > +# no stale port. Yes this looks good to me! Thanks, Eelco _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev