On 7/19/22 19:19, Numan Siddique wrote:
> On Thu, Jul 7, 2022 at 6:27 AM Lucas Alvares Gomes
> <[email protected]> wrote:
>>
>> On Thu, Jul 7, 2022 at 12:24 PM Lucas Alvares Gomes
>> <[email protected]> wrote:
>>>
>>> Hi,
>>>
>>> I tested this series of patches with OpenStack upstream [0] since we
>>> have some automated tests for IGMP and it seems good:
>>>
>>> test_multicast_between_vms_on_same_network[id-113486fc-24c9-4be4-8361-03b1c9892867]
>>> pass (link: 
>>> https://98794ab76263ee253bc7-6b6bf6aa2495d781bdba3c8c61916451.ssl.cf1.rackcdn.com/848934/2/check/neutron-tempest-plugin-ovn/b3191bf/testr_results.html,
>>> this will expire at some point)
>>>
>>> What I did was to change ML2/OVN to set mcast_flood to False on the
>>> LSPs, apply the series of patches in OVN and then run the tests.
>>>
>>> Tested-By: 
>>> https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/848934/
>>>
>>
>> Tested-By: Lucas Alvares Gomes <[email protected]>
> 
> Acked-by: Numan Siddique <[email protected]>

Thanks, I included your ack in v2 (no change to this patch):
https://patchwork.ozlabs.org/project/ovn/list/?series=310294&state=*

> 
> It would be great if someone else also reviews this patch.
> 

I agree, the more eyes on this change the better!

> Numan

Thanks,
Dumitru

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to