[ovs-dev] [PATCH v4 ovn 0/3] controller: Notify when an OVN port is ready to use.
Dumitru Ceara
dceara at redhat.com
Wed Jan 13 15:05:21 UTC 2021
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!
More information about the dev
mailing list