[ovs-git] [ovn-org/ovn] 27064c: ovn-northd: Avoid verification of NB_Global.option...

Ilya Maximets noreply at github.com
Mon Sep 13 18:48:11 UTC 2021


  Branch: refs/heads/branch-21.06
  Home:   https://github.com/ovn-org/ovn
  Commit: 27064c554fe6a3dfe4dd5800e884eec70cd3d793
      https://github.com/ovn-org/ovn/commit/27064c554fe6a3dfe4dd5800e884eec70cd3d793
  Author: Ilya Maximets <i.maximets at ovn.org>
  Date:   2021-09-13 (Mon, 13 Sep 2021)

  Changed paths:
    M northd/ovn-northd.c

  Log Message:
  -----------
  ovn-northd: Avoid verification of NB_Global.options if nothing changed.

northd constructs transaction for the Northbound database options even
if they don't need to be changed.  This also includes verification of
a current state.  IDL will eventually drop the transaction if it will
not contain any other operations, but if there are some other
operations, this useless update to the same value will be included
along with the verification.  This causes transaction failures because
NB_Global.options can be updated by CMS at the same time.

For example, ovn-kubernetes sets 'options:e2e_timestamp' for it's own
purposes, and if this value will be updated while there is an in-flight
transaction from the northd, transaction will fail the verification and
northd will have to re-try it.

To avoid these issues, updating and verifying options only if needed.

Fixes: b07f1bc3d068 ("Add VXLAN support for non-VTEP datapath bindings")
Signed-off-by: Ilya Maximets <i.maximets at ovn.org>
Acked-by: Mark D. Gray <mark.d.gray at redhat.com>
Signed-off-by: Mark Michelson <mmichels at redhat.com>




More information about the git mailing list