Re: [ovs-dev] [PATCH ovn v2] northd: Fix address set incremental processing

2023-06-07 Thread Ales Musil
On Tue, Jun 6, 2023 at 10:17 PM Han Zhou wrote: > > > On Thu, Jun 1, 2023 at 2:55 PM Han Zhou wrote: > > > > > > > > On Wed, May 31, 2023 at 11:16 PM Ales Musil wrote: > > > > > > > > > > > > On Thu, Jun 1, 2023 at 7:44 AM Han Zhou wrote: > > >> > > >> > > >> > > >> On Wed, May 24, 2023 at

Re: [ovs-dev] [PATCH ovn v2] northd: Fix address set incremental processing

2023-06-06 Thread Han Zhou
On Thu, Jun 1, 2023 at 2:55 PM Han Zhou wrote: > > > > On Wed, May 31, 2023 at 11:16 PM Ales Musil wrote: > > > > > > > > On Thu, Jun 1, 2023 at 7:44 AM Han Zhou wrote: > >> > >> > >> > >> On Wed, May 24, 2023 at 11:27 PM Ales Musil wrote: > >> > > >> > The incremental processing is broken for

Re: [ovs-dev] [PATCH ovn v2] northd: Fix address set incremental processing

2023-06-01 Thread Han Zhou
On Wed, May 31, 2023 at 11:16 PM Ales Musil wrote: > > > > On Thu, Jun 1, 2023 at 7:44 AM Han Zhou wrote: >> >> >> >> On Wed, May 24, 2023 at 11:27 PM Ales Musil wrote: >> > >> > The incremental processing is broken for addresses >> > that have mask which could "erase" portion of the address >>

Re: [ovs-dev] [PATCH ovn v2] northd: Fix address set incremental processing

2023-06-01 Thread Ales Musil
On Thu, Jun 1, 2023 at 7:44 AM Han Zhou wrote: > > > On Wed, May 24, 2023 at 11:27 PM Ales Musil wrote: > > > > The incremental processing is broken for addresses > > that have mask which could "erase" portion of the address > > itself e.g. 10.16.0.47/4, after applying the mask with token > >

Re: [ovs-dev] [PATCH ovn v2] northd: Fix address set incremental processing

2023-05-31 Thread Han Zhou
On Wed, May 24, 2023 at 11:27 PM Ales Musil wrote: > > The incremental processing is broken for addresses > that have mask which could "erase" portion of the address > itself e.g. 10.16.0.47/4, after applying the mask with token > parser the address becomes 0.0.0.0/4, which is fine for > logical

[ovs-dev] [PATCH ovn v2] northd: Fix address set incremental processing

2023-05-25 Thread Ales Musil
The incremental processing is broken for addresses that have mask which could "erase" portion of the address itself e.g. 10.16.0.47/4, after applying the mask with token parser the address becomes 0.0.0.0/4, which is fine for logical flows. However, for the deletion/addition to database we need