Re: [ovs-dev] [PATCH ovn] controller: Don't allocate zone ids for non-VIF port bindings.

2021-08-23 Thread Numan Siddique
On Mon, Aug 23, 2021 at 11:26 AM Vladislav Odintsov wrote: > > Hi Numan, > thanks for the quick fix. This worked for me. > > Regards, > Vladislav Odintsov > > > On 23 Aug 2021, at 18:24, Vladislav Odintsov wrote: > > > > Tested-by: Vladislav Odintsov mailto:odiv...@gmail.com>> Thanks for

Re: [ovs-dev] [PATCH ovn] controller: Don't allocate zone ids for non-VIF port bindings.

2021-08-23 Thread Vladislav Odintsov
Hi Numan, thanks for the quick fix. This worked for me. Regards, Vladislav Odintsov > On 23 Aug 2021, at 18:24, Vladislav Odintsov wrote: > > Tested-by: Vladislav Odintsov mailto:odiv...@gmail.com>> > > Regards, > Vladislav Odintsov > >> On 21 Aug 2021, at 07:58, num...@ovn.org

Re: [ovs-dev] [PATCH ovn] controller: Don't allocate zone ids for non-VIF port bindings.

2021-08-23 Thread Vladislav Odintsov
Tested-by: Vladislav Odintsov Regards, Vladislav Odintsov > On 21 Aug 2021, at 07:58, num...@ovn.org wrote: > > From: Numan Siddique > > The commit 6fb87aad8c3("controller: Improve ct zone handling.") > caused a regression. After this commit ovn-controller is allocating > zone id for non

[ovs-dev] [PATCH ovn] controller: Don't allocate zone ids for non-VIF port bindings.

2021-08-20 Thread numans
From: Numan Siddique The commit 6fb87aad8c3("controller: Improve ct zone handling.") caused a regression. After this commit ovn-controller is allocating zone id for non VIF port bindings and this is causing unexpected behavior. This patch fixes this issue. Also added a test case to cover this