Re: [ovs-dev] [PATCH ovn] northd.c: Incremental processing for first/last switch port change.

2023-09-01 Thread Han Zhou
On Thu, Aug 31, 2023 at 12:47 PM Mark Michelson wrote: > > Hi Han, thanks for the change. > > Acked-by: Mark Michelson Thanks Mark and Ales. I applied it to main with Ales's comment addressed. Han > > On 8/17/23 12:14, Han Zhou wrote: > > On Thu, Aug 17, 2023 at 2:13 AM Ales Musil wrote: >

Re: [ovs-dev] [PATCH ovn] northd.c: Incremental processing for first/last switch port change.

2023-08-31 Thread Mark Michelson
Hi Han, thanks for the change. Acked-by: Mark Michelson On 8/17/23 12:14, Han Zhou wrote: On Thu, Aug 17, 2023 at 2:13 AM Ales Musil wrote: On Wed, Aug 16, 2023 at 2:56 AM Han Zhou wrote: Commit 1eb09838 fixed an incremental processing problem by falling back to recompute. The problem

Re: [ovs-dev] [PATCH ovn] northd.c: Incremental processing for first/last switch port change.

2023-08-17 Thread Han Zhou
On Thu, Aug 17, 2023 at 2:13 AM Ales Musil wrote: > > > > On Wed, Aug 16, 2023 at 2:56 AM Han Zhou wrote: >> >> Commit 1eb09838 fixed an incremental processing problem by falling back >> to recompute. The problem was handling VIF changes for a LS that >> has router ports when: >> - adding the

Re: [ovs-dev] [PATCH ovn] northd.c: Incremental processing for first/last switch port change.

2023-08-17 Thread Ales Musil
On Wed, Aug 16, 2023 at 2:56 AM Han Zhou wrote: > Commit 1eb09838 fixed an incremental processing problem by falling back > to recompute. The problem was handling VIF changes for a LS that > has router ports when: > - adding the first switch port to the LS, or > - deleting the last switch port

[ovs-dev] [PATCH ovn] northd.c: Incremental processing for first/last switch port change.

2023-08-15 Thread Han Zhou
Commit 1eb09838 fixed an incremental processing problem by falling back to recompute. The problem was handling VIF changes for a LS that has router ports when: - adding the first switch port to the LS, or - deleting the last switch port from the LS because there are router port related lflows that