Re: [ovs-dev] [PATCH ovn 1/2] tests: Change ct zone UUID->name migration test to use GW routers.

2023-08-15 Thread Dumitru Ceara
On 8/14/23 00:23, Han Zhou wrote: > On Fri, Aug 11, 2023 at 5:24 AM Dumitru Ceara wrote: >> >> The test added by 5a1d82cb28c5 ("controller: Migrate from ct zone UUID >> name to component name") was, in theory, correct. However, due to the >> way conditional monitoring works, the test cannot pass

Re: [ovs-dev] [PATCH ovn 1/2] tests: Change ct zone UUID->name migration test to use GW routers.

2023-08-13 Thread Ales Musil
On Fri, Aug 11, 2023 at 2:24 PM Dumitru Ceara wrote: > > The test added by 5a1d82cb28c5 ("controller: Migrate from ct zone UUID > name to component name") was, in theory, correct. However, due to the > way conditional monitoring works, the test cannot pass on deployments > where distributed route

Re: [ovs-dev] [PATCH ovn 1/2] tests: Change ct zone UUID->name migration test to use GW routers.

2023-08-13 Thread Han Zhou
On Fri, Aug 11, 2023 at 5:24 AM Dumitru Ceara wrote: > > The test added by 5a1d82cb28c5 ("controller: Migrate from ct zone UUID > name to component name") was, in theory, correct. However, due to the > way conditional monitoring works, the test cannot pass on deployments > where distributed route

[ovs-dev] [PATCH ovn 1/2] tests: Change ct zone UUID->name migration test to use GW routers.

2023-08-11 Thread Dumitru Ceara
The test added by 5a1d82cb28c5 ("controller: Migrate from ct zone UUID name to component name") was, in theory, correct. However, due to the way conditional monitoring works, the test cannot pass on deployments where distributed routers are used. These will initially not be considered "local" dat