[ovs-dev] [PATCH v4 ovn 0/3] controller: Notify when an OVN port is ready to use.

[Dumitru Ceara]

On 1/13/21 3:07 PM, Numan Siddique wrote:
> 
> 
> On Wed, Jan 13, 2021 at 2:53 PM Dumitru Ceara <dceara at redhat.com
> <mailto:dceara at redhat.com>> wrote:
> 
>     This series adds support to ovn-controller to notify northd that all OVS
>     flows and OVS interface corresponding to an OVN logical port have been
>     installed in OVS.
> 
>     The series extends the ofctrl barrier implementation and makes it
>     generic
>     so that multiple modules can use it.  For now these are:
>     - main ovn-controller for nb_cfg propagation.
>     - binding module for marking Port_Bindings as installed.
> 
>     V4:
>     - Added acks from Mark and Numan.
>     - Address Numan's comments:
>       - Patch2:
>         - Reorganize ofctrl-seqno.[ch] according to coding guidelines.
>       - Patch3:
>         - Fix RBAC issue reported by Numan.
>     V3:
>     - Address Mark's comments:
>       - Patch2:
>         - Remove unnecessary ENABLE_UNIT_TESTS.
>       - Patch3:
>         - Add test to check that the ovn-installed external-id is
>     propagated to
>           OVS.  The test also checks that the SB Port_Binding.up and
>           NB.Logical_Switch_Port.up fields are properly set.
>     - Patch3:
>       - Improve the checks in binding_seqno_install() to cleanup stale
>     bindings
>         early.
>     V2:
>     - Patch 3:
>       - Fix check in binding_seqno_install() to take into account deleted
>         port bindings.
> 
>     Dumitru Ceara (3):
>           ofctrl: Rename 'nb_cfg' to 'req_cfg'.
>           controller: Implement a generic barrier based on ofctrl
>     cur_cfg sync.
>           binding: Set Logical_Switch_Port.up when all OVS flows are
>     installed.
> 
> 
> Thanks Dumitru for v4 and Mark for reviews.
> 
> I applied this series to master.
> 

Thanks!