[ovs-dev] [PATCH ovn v9 7/7] ovn-northd-ddlog: New implementation of ovn-northd based on ddlog.

Dumitru Ceara dceara at redhat.com
Wed Feb 17 16:40:15 UTC 2021


On 2/17/21 2:49 AM, Ben Pfaff wrote:
> On Fri, Dec 11, 2020 at 03:43:22PM +0100, Dumitru Ceara wrote:
>>> diff --git a/tests/ovn.at b/tests/ovn.at
>>> index 6e396895826f..2e4eaf7f12f7 100644
>>> --- a/tests/ovn.at
>>> +++ b/tests/ovn.at
>>> @@ -16982,6 +16982,10 @@ AT_CLEANUP
>>>   
>>>   OVN_FOR_EACH_NORTHD([
>>>   AT_SETUP([ovn -- IGMP snoop/querier/relay])
>>> +
>>> +dnl This test has problems with ovn-northd-ddlog.
>>> +AT_SKIP_IF([test NORTHD_TYPE = ovn-northd-ddlog && test "$RUN_ANYWAY" != yes])
>>
>> I might be wrong but I think the ddlog part still misses the logic from
>> this commit from the ovn-northd C implementation:
>>
>> https://github.com/ovn-org/ovn/commit/97778ab3e422ac071faa67f9f477fd54977e9c04
> 
> Thanks very much, incorporating that fixed the IGMP and MLD
> snoop/querier/relay tests.

No problem.

> 
> The interconnection test still fails, probably because I haven't
> identified what's missing in that case yet.  Any insight is really
> welcome.
> 

I don't think there were any interconnection related changes that went 
into ovn-northd recently but maybe Han is the better person to ask.

Regards,
Dumitru



More information about the dev mailing list