[ovs-git] [ovn-org/ovn] 2f933f: ofctrl: Rename 'nb_cfg' to 'req_cfg'.

Dumitru Ceara noreply at github.com
Wed Jan 13 14:06:29 UTC 2021


  Branch: refs/heads/master
  Home:   https://github.com/ovn-org/ovn
  Commit: 2f933fc560330022cfc816ed870da6e5847809c9
      https://github.com/ovn-org/ovn/commit/2f933fc560330022cfc816ed870da6e5847809c9
  Author: Dumitru Ceara <dceara at redhat.com>
  Date:   2021-01-13 (Wed, 13 Jan 2021)

  Changed paths:
    M controller/ofctrl.c
    M controller/ofctrl.h
    M controller/ovn-controller.c

  Log Message:
  -----------
  ofctrl: Rename 'nb_cfg' to 'req_cfg'.

A future commit will extend the ofctrl OVS barrier sync mechanism to
make it usable by multiple components.  One of the use cases will be
'nb_cfg' sync but it may not be the only one.

Signed-off-by: Dumitru Ceara <dceara at redhat.com>
Acked-by: Mark Michelson <mmichels at redhat.com>
Acked-by: Numan Siddique <numans at ovn.org>
Signed-off-by: Numan Siddique <numans at ovn.org>


  Commit: c93c626248c120eeaffafd323aef323d3b2507ab
      https://github.com/ovn-org/ovn/commit/c93c626248c120eeaffafd323aef323d3b2507ab
  Author: Dumitru Ceara <dceara at redhat.com>
  Date:   2021-01-13 (Wed, 13 Jan 2021)

  Changed paths:
    M controller/automake.mk
    A controller/ofctrl-seqno.c
    A controller/ofctrl-seqno.h
    M controller/ovn-controller.c
    A controller/test-ofctrl-seqno.c
    M tests/automake.mk
    A tests/ovn-ofctrl-seqno.at
    M tests/testsuite.at

  Log Message:
  -----------
  controller: Implement a generic barrier based on ofctrl cur_cfg sync.

A new module, 'ofctrl-seqno', is added to implement this generic
barrier.  Other modules can register their own types of seqno update
requests.  The barrier implementation ensures that the a seqno update
request is acked (returned by ofctrl_acked_seqnos_get()) only if the
OVS flow operations that have been requested when the seqno update
request was queued have been processed by OVS.

For now, the only user of this barrier is the main ovn-controller
module but a future commit will use it too in order to mark
Port_Bindings and OVS interfaces as "fully installed".

This commit also adds unit tests for the new 'ofctrl-seqno' module.
The unit test structure is inspired by Mark Michelson's patch:
http://patchwork.ozlabs.org/project/ovn/patch/20201216182421.234772-3-mmichels@redhat.com/

Signed-off-by: Dumitru Ceara <dceara at redhat.com>
Acked-by: Mark Michelson <mmichels at redhat.com>
Signed-off-by: Numan Siddique <numans at ovn.org>


  Commit: 4d3cb42b076bb58fd8f01ab8ad146ffd539f2152
      https://github.com/ovn-org/ovn/commit/4d3cb42b076bb58fd8f01ab8ad146ffd539f2152
  Author: Dumitru Ceara <dceara at redhat.com>
  Date:   2021-01-13 (Wed, 13 Jan 2021)

  Changed paths:
    M NEWS
    M controller/binding.c
    M controller/binding.h
    M controller/ovn-controller.c
    M northd/ovn-northd.c
    M ovn-sb.ovsschema
    M ovn-sb.xml
    M tests/ovn.at
    M utilities/ovn-sbctl.c

  Log Message:
  -----------
  binding: Set Logical_Switch_Port.up when all OVS flows are installed.

Using the ofctrl-seqno generic barrier, register a new type of
notifications for Port_Bindings.  This allows us to delay setting
the Logical_Switch_Port.up field until all OVS flows corresponding
to the logical port and underlying OVS interface have been installed.

This commit also marks the OVS interface as "installed by OVN" by
setting a new "ovn-installed" external-id in the OVS Interface record
when the port is fully wired by OVN.

Signed-off-by: Dumitru Ceara <dceara at redhat.com>
Acked-by: Mark Michelson <mmichels at redhat.com>
Signed-off-by: Numan Siddique <numans at ovn.org>


Compare: https://github.com/ovn-org/ovn/compare/ecc7bd74d8c0...4d3cb42b076b


More information about the git mailing list