On 08/09/2021 13:18, Ilya Maximets wrote:
> 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 <[email protected]>
> ---
Acked-by: Mark D. Gray <[email protected]>
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
