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

2021-02-17 Thread Dumitru Ceara
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([

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

2021-02-17 Thread Dumitru Ceara
On 2/16/21 10:16 PM, Ben Pfaff wrote: On Fri, Dec 11, 2020 at 03:43:22PM +0100, Dumitru Ceara wrote: There are a few tests that fail (also northd-C tests) and I suspect it's because the OVS IDL split patches [0] this commit depends on don't include some of the recent fixes that went into the

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

2021-02-16 Thread Ben Pfaff
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

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

2021-02-16 Thread Ben Pfaff
On Fri, Dec 11, 2020 at 03:43:22PM +0100, Dumitru Ceara wrote: > There are a few tests that fail (also northd-C tests) and I suspect it's > because the OVS IDL split patches [0] this commit depends on don't > include some of the recent fixes that went into the OVS IDL code, at > least the

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

2021-02-02 Thread Ben Pfaff
On Fri, Dec 11, 2020 at 03:43:22PM +0100, Dumitru Ceara wrote: > Also, I'm still seeing the libopenvswitchavx512.la related error we > reported earlier: > > error: failed to run `rustc` to learn about target-specific information > > Caused by: > process didn't exit successfully: `rustc -

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

2020-12-11 Thread Dumitru Ceara
On 12/2/20 7:50 AM, Ben Pfaff wrote: > From: Leonid Ryzhyk > > 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.

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

2020-12-02 Thread 0-day Robot
Bleep bloop. Greetings Ben Pfaff, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: Comment with 'xxx' marker #3534 FILE: northd/ovn-northd-ddlog.c:164: .cs =