[ovs-git] [ovn-org/ovn] 37f202: ovn-nbctl: Fix lrp-set-gateway-chassis.

Han Zhou noreply at github.com
Thu Nov 11 22:57:29 UTC 2021


  Branch: refs/heads/branch-21.09
  Home:   https://github.com/ovn-org/ovn
  Commit: 37f2027c3c4af42bc4d98fdc81ed0b65fbde2725
      https://github.com/ovn-org/ovn/commit/37f2027c3c4af42bc4d98fdc81ed0b65fbde2725
  Author: Han Zhou <hzhou at ovn.org>
  Date:   2021-11-11 (Thu, 11 Nov 2021)

  Changed paths:
    M tests/ovn-nbctl.at
    M utilities/ovn-nbctl.c

  Log Message:
  -----------
  ovn-nbctl: Fix lrp-set-gateway-chassis.

Currently this command assumes that if the gateway_chassis record with
expected name exists it is set to the logical port, so once the record
is found it not set to the lrp again. However, this assumption is not
always true.

An example is that when combined with a lrp-del and then lrp-add
commands before lrp-set-gateway-chassis in the same transaction, the
gateway_chassis record will be found but it is not set to the lrp. This
is causing the gateway-chassis setting flapping in ovn-kubernetes'
cluster router logical ports.

This patch makes sure the gateway_chassis record (existed or newly
created) is set to the lrp's gateway-chassis column.

Signed-off-by: Han Zhou <hzhou at ovn.org>
Acked-by: Numan Siddique <numans at ovn.org>
Acked-by: Mark Michelson <mmichels at redhat.com>
(cherry picked from commit e6bcb889a6b324f80b120a113a0eecbc99ac9166)




More information about the git mailing list