[ovs-dev] [PATCH ovn v2] ovn-controller: Fix incremental processing of Port_Binding deletes.

[Dumitru Ceara]

On 8/31/20 4:09 PM, Numan Siddique wrote:
> 
> 
> On Mon, Aug 31, 2020 at 5:45 PM Dumitru Ceara <dceara at redhat.com
> <mailto:dceara at redhat.com>> wrote:
> 
>     If a Port_Binding is deleted from the Southbound DB and the
>     corresponding OVS interface is also deleted from the OVS DB, and
>     if both updates are received and processed by ovn-controller in
>     the same iteration, ovn-controller should process port_binding
>     delete operations first.
> 
>     This commit also adds three new unixctl debug commands for
>     ovn-controller:
>     - debug/pause: pause ovn-controller processing, except unixctl commands.
>     - debug/resume: resume ovn-controller processing.
>     - debug/status: return the status of the ovn-controller processing.
> 
>     These new commands are needed by the test for this scenario as without
>     them we have no way of ensuring predictable results. Users should not
>     use these commands in production. This is also why the commands are not
>     documented.
> 
>     CC: Numan Siddique <numans at ovn.org <mailto:numans at ovn.org>>
>     Fixes: 6b0f01116bab ("ovn-controller: Handle runtime data changes in
>     flow output engine")
>     Reported-by: Tim Rozet <trozet at redhat.com <mailto:trozet at redhat.com>>
>     Reported-at: https://bugzilla.redhat.com/1871961
>     Signed-off-by: Dumitru Ceara <dceara at redhat.com
>     <mailto:dceara at redhat.com>>
>     ---
>     v2: Rework the fix per Numan's suggestion and run the port_binding
>         handler before the ovs iface handler.
>     ---
> 
> 
> Thanks Dumitru for v2.
> 
> I applied this patch to master and branch-20.06.
> 
> Numan
>  

Thanks!