Re: [ovs-dev] [PATCH] ovn-controller: Fix empty address set parsing problem.

2017-09-11 Thread Ben Pfaff
On Sun, Sep 10, 2017 at 11:10:58AM -0700, Han Zhou wrote: > On Sun, Sep 10, 2017 at 10:56 AM, Ben Pfaff wrote: > > > > On Sat, Sep 09, 2017 at 10:58:25PM -0700, Han Zhou wrote: > > > When an address set is empty, current implementation will generate > > > an ovs flow that matches

Re: [ovs-dev] [PATCH] ovn-controller: Fix empty address set parsing problem.

2017-09-10 Thread Han Zhou
On Sun, Sep 10, 2017 at 10:56 AM, Ben Pfaff wrote: > > On Sat, Sep 09, 2017 at 10:58:25PM -0700, Han Zhou wrote: > > When an address set is empty, current implementation will generate > > an ovs flow that matches random things (and in most cases matching > > everything) due to a

Re: [ovs-dev] [PATCH] ovn-controller: Fix empty address set parsing problem.

2017-09-10 Thread Ben Pfaff
On Sat, Sep 09, 2017 at 10:58:25PM -0700, Han Zhou wrote: > When an address set is empty, current implementation will generate > an ovs flow that matches random things (and in most cases matching > everything) due to a problem in expression parser of constant set. > This patch fixes it by

[ovs-dev] [PATCH] ovn-controller: Fix empty address set parsing problem.

2017-09-09 Thread Han Zhou
When an address set is empty, current implementation will generate an ovs flow that matches random things (and in most cases matching everything) due to a problem in expression parser of constant set. This patch fixes it by replacing the expression by a boolean false when the set is empty, and