Re: [ovs-dev] [PATCH ovn] Exclude inport and outport symbol tables from conjunction

2019-09-19 Thread Han Zhou
On Wed, Sep 18, 2019 at 2:51 AM Dumitru Ceara wrote: > > On Wed, Sep 18, 2019 at 9:30 AM Dumitru Ceara wrote: > > > > On Tue, Sep 17, 2019 at 6:49 PM Han Zhou wrote: > > > > > > > > > > > > On Tue, Sep 17, 2019 at 5:21 AM Mark Michelson wrote: > > > > > > > > On 9/16/19 12:04 PM, Han Zhou wrote

Re: [ovs-dev] [PATCH ovn] Exclude inport and outport symbol tables from conjunction

2019-09-18 Thread Dumitru Ceara
On Wed, Sep 18, 2019 at 9:30 AM Dumitru Ceara wrote: > > On Tue, Sep 17, 2019 at 6:49 PM Han Zhou wrote: > > > > > > > > On Tue, Sep 17, 2019 at 5:21 AM Mark Michelson wrote: > > > > > > On 9/16/19 12:04 PM, Han Zhou wrote: > > > > > > > > > > > > On Mon, Sep 16, 2019 at 4:15 AM Dumitru Ceara >

Re: [ovs-dev] [PATCH ovn] Exclude inport and outport symbol tables from conjunction

2019-09-18 Thread Dumitru Ceara
On Tue, Sep 17, 2019 at 6:49 PM Han Zhou wrote: > > > > On Tue, Sep 17, 2019 at 5:21 AM Mark Michelson wrote: > > > > On 9/16/19 12:04 PM, Han Zhou wrote: > > > > > > > > > On Mon, Sep 16, 2019 at 4:15 AM Dumitru Ceara > > > wrote: > > > > > > > > On Sat, Sep 14, 2019

Re: [ovs-dev] [PATCH ovn] Exclude inport and outport symbol tables from conjunction

2019-09-17 Thread Han Zhou
On Tue, Sep 17, 2019 at 5:21 AM Mark Michelson wrote: > > On 9/16/19 12:04 PM, Han Zhou wrote: > > > > > > On Mon, Sep 16, 2019 at 4:15 AM Dumitru Ceara > > wrote: > > > > > > On Sat, Sep 14, 2019 at 7:16 PM Han Zhou > > wrote: > > > > > >

Re: [ovs-dev] [PATCH ovn] Exclude inport and outport symbol tables from conjunction

2019-09-17 Thread Mark Michelson
On 9/16/19 12:04 PM, Han Zhou wrote: On Mon, Sep 16, 2019 at 4:15 AM Dumitru Ceara > wrote: > > On Sat, Sep 14, 2019 at 7:16 PM Han Zhou > wrote: > > > > > > > > On Sat, Sep 14, 2019 at 9:09 AM Han Zhou > wrot

Re: [ovs-dev] [PATCH ovn] Exclude inport and outport symbol tables from conjunction

2019-09-16 Thread Han Zhou
On Mon, Sep 16, 2019 at 4:15 AM Dumitru Ceara wrote: > > On Sat, Sep 14, 2019 at 7:16 PM Han Zhou wrote: > > > > > > > > On Sat, Sep 14, 2019 at 9:09 AM Han Zhou wrote: > > > > > > > > > > > > On Sat, Sep 14, 2019 at 12:40 AM Numan Siddique wrote: > > > > > > > > > > > > > > > > On Sat, Sep 14,

Re: [ovs-dev] [PATCH ovn] Exclude inport and outport symbol tables from conjunction

2019-09-16 Thread Dumitru Ceara
On Sat, Sep 14, 2019 at 7:16 PM Han Zhou wrote: > > > > On Sat, Sep 14, 2019 at 9:09 AM Han Zhou wrote: > > > > > > > > On Sat, Sep 14, 2019 at 12:40 AM Numan Siddique wrote: > > > > > > > > > > > > On Sat, Sep 14, 2019 at 2:41 AM Daniel Alvarez Sanchez > > > wrote: > > >> > > >> Acked-by: Dan

Re: [ovs-dev] [PATCH ovn] Exclude inport and outport symbol tables from conjunction

2019-09-14 Thread Han Zhou
On Sat, Sep 14, 2019 at 9:09 AM Han Zhou wrote: > > > > On Sat, Sep 14, 2019 at 12:40 AM Numan Siddique wrote: > > > > > > > > On Sat, Sep 14, 2019 at 2:41 AM Daniel Alvarez Sanchez < dalva...@redhat.com> wrote: > >> > >> Acked-by: Daniel Alvarez > >> > >> > >> On Fri, Sep 13, 2019 at 11:02 PM M

Re: [ovs-dev] [PATCH ovn] Exclude inport and outport symbol tables from conjunction

2019-09-14 Thread Han Zhou
On Sat, Sep 14, 2019 at 12:40 AM Numan Siddique wrote: > > > > On Sat, Sep 14, 2019 at 2:41 AM Daniel Alvarez Sanchez < dalva...@redhat.com> wrote: >> >> Acked-by: Daniel Alvarez >> >> >> On Fri, Sep 13, 2019 at 11:02 PM Mark Michelson wrote: >> > >> > Acked-by: Mark Michelson >> > >> > It sucks

Re: [ovs-dev] [PATCH ovn] Exclude inport and outport symbol tables from conjunction

2019-09-14 Thread Numan Siddique
On Sat, Sep 14, 2019 at 2:41 AM Daniel Alvarez Sanchez wrote: > Acked-by: Daniel Alvarez > > > On Fri, Sep 13, 2019 at 11:02 PM Mark Michelson > wrote: > > > > Acked-by: Mark Michelson > > > > It sucks that we lose the efficiency of the conjunctive match altogether > > on port groups because of

Re: [ovs-dev] [PATCH ovn] Exclude inport and outport symbol tables from conjunction

2019-09-13 Thread Daniel Alvarez Sanchez
Acked-by: Daniel Alvarez On Fri, Sep 13, 2019 at 11:02 PM Mark Michelson wrote: > > Acked-by: Mark Michelson > > It sucks that we lose the efficiency of the conjunctive match altogether > on port groups because of this error, but I understand this is a huge > bug and needs fixing. If I'm not mi

Re: [ovs-dev] [PATCH ovn] Exclude inport and outport symbol tables from conjunction

2019-09-13 Thread Mark Michelson
Acked-by: Mark Michelson It sucks that we lose the efficiency of the conjunctive match altogether on port groups because of this error, but I understand this is a huge bug and needs fixing. Perhaps it would be good to start up a discussion on this list about a more longterm solution that wou

[ovs-dev] [PATCH ovn] Exclude inport and outport symbol tables from conjunction

2019-09-13 Thread nusiddiq
From: Numan Siddique If there are multiple ACLs associated with a port group and they match on a range of some field, then ovn-controller doesn't install the flows properly and this results in broken ACL functionality. For example, if there is a port group - pg1 with logical ports - [p1, p2] and