It was great fun, I haven't had a nice satisfying bug hunt in a while.
On Fri, May 14, 2021 at 03:39:22PM -0400, Ihar Hrachyshka wrote: > Thank you Ben, > > an impressive feat and an interesting write-up. :) > > Ihar > > On Fri, May 14, 2021 at 12:21 AM Ben Pfaff <[email protected]> wrote: > > > > On Wed, May 12, 2021 at 09:21:32PM -0400, Ihar Hrachyshka wrote: > > > Hi Numan, > > > > > > both 3187b9fef124e038e474270a2728fe94bdca8eef (ovn-northd: introduce > > > new allow-stateless ACL verb) and > > > 127bf166ccf4a2509f670c48a00b0340039f20d2 (northd: Support flow > > > offloading for logical switches with no ACLs.) got merged in upstream > > > master, and this combination broke the following tests: > > > > > > 774: ovn -- ACL allow-stateless omit conntrack - Port_Group -- > > > ovn-northd-ddlog -- dp-groups=yes FAILED (ovn-northd.at:2752) > > > 775: ovn -- ACL allow-stateless omit conntrack - Port_Group -- > > > ovn-northd-ddlog FAILED (ovn-northd.at:2752) > > > > > > while the other scenarios are passing: > > > > > > 768: ovn -- ACL allow-stateless omit conntrack - Logical_Switch -- > > > ovn-northd -- dp-groups=yes ok > > > 769: ovn -- ACL allow-stateless omit conntrack - Logical_Switch -- > > > ovn-northd ok > > > 770: ovn -- ACL allow-stateless omit conntrack - Logical_Switch -- > > > ovn-northd-ddlog -- dp-groups=yes ok > > > 771: ovn -- ACL allow-stateless omit conntrack - Logical_Switch -- > > > ovn-northd-ddlog ok > > > 772: ovn -- ACL allow-stateless omit conntrack - Port_Group -- > > > ovn-northd -- dp-groups=yes ok > > > 773: ovn -- ACL allow-stateless omit conntrack - Port_Group -- ovn-northd > > > ok > > > > This turned to out to be an all-afternoon-and-evening saga, but I got it > > fixed and documented: > > https://mail.openvswitch.org/pipermail/ovs-dev/2021-May/382993.html > > https://github.com/vmware/differential-datalog/pull/977 > > > _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
