On Thu, Nov 19, 2020 at 09:11:38AM +0100, Dumitru Ceara wrote: > On 11/19/20 6:13 AM, Ben Pfaff wrote: > > From: Leonid Ryzhyk <lryz...@vmware.com> > > > > This implementation is incremental, meaning that it only recalculates > > what is needed for the southbound database when northbound changes > > occur. It is expected to scale better than the C implementation, > > for large deployments. (This may take testing and tuning to be > > effective.) > > > > There are three tests that I'm having mysterious trouble getting > > to work with DDlog. For now, I've marked the testsuite to skip > > them unless RUN_ANYWAY=yes is set in the environment. > > > > Signed-off-by: Leonid Ryzhyk <lryz...@vmware.com> > > Co-authored-by: Justin Pettit <jpet...@ovn.org> > > Signed-off-by: Justin Pettit <jpet...@ovn.org> > > Co-authored-by: Ben Pfaff <b...@ovn.org> > > Signed-off-by: Ben Pfaff <b...@ovn.org> > > Acked-by: Dumitru Ceara <dce...@redhat.com> > > Hi Ben, > > This is not a review, just a note that my "Acked-by" should probably go > to patch 2/5 "tests: Prepare for multiple northd types.". Unless I'm > mistaken, I didn't review the last patch of the series yet, except the > fact that I tried running the OVN tests. > > I'm not saying patch 5/5 "ovn-northd-ddlog: New implementation of > ovn-northd based on ddlog." is not good, just that I didn't review it yet :)
Thanks, I misunderstood. I'll move the ack. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
