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

[Ben Pfaff]

On Mon, Nov 09, 2020 at 02:44:53PM +0100, Dumitru Ceara wrote:
> There are a few others that are ddlog-related:
> 
> 145: ovn -- /32 router IP address -- ovn-northd-ddlog FAILED
> (ovs-macros.at:231)
> 
> This is because the following logical flows are not added to the SB by
> ovn-northd-ddlog:
> table=14(ls_in_arp_rsp      ), priority=100  , match=(arp.tpa ==
> 10.0.0.2 && arp.op == 1 && inport == "alice1"), action=(next;)
> table=14(ls_in_arp_rsp      ), priority=50   , match=(arp.tpa ==
> 10.0.0.2 && arp.op == 1), action=(eth.dst = eth.src; eth.src =
> f0:00:00:01:02:04; arp.op = 2; /* ARP reply */ arp.tha = arp.sha;
> arp.sha = f0:00:00:01:02:04; arp.tpa = arp.spa; arp.spa = 10.0.0.2;
> outport = inport; flags.loopback = 1; output;)

Hmm, I don't see this failure, and the diff I see between the flow
tables is empty if I filter uuids and drop the inconsistent use of /32
suffixes, e.g.:

    diff -u <(uuidfilt tests/testsuite.dir/144/sbflows | sed 's,/32,,g') <(uuidfilt tests/testsuite.dir/145/sbflows | sed 's,/32,,g')

I wonder why we see something different, is this racy for you?