[ovs-git] [openvswitch/ovs] cd37e7: ovn-northd: Refactor the code which sets nat_addre...

numansiddique noreply at github.com
Wed Jul 3 20:25:28 UTC 2019


  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: cd37e73414ddba8e6d80c237cee3daee3f3dd6d1
      https://github.com/openvswitch/ovs/commit/cd37e73414ddba8e6d80c237cee3daee3f3dd6d1
  Author: Numan Siddique <nusiddiq at redhat.com>
  Date:   2019-07-03 (Wed, 03 Jul 2019)

  Changed paths:
    M ovn/northd/ovn-northd.c

  Log Message:
  -----------
  ovn-northd: Refactor the code which sets nat_addresses

The present code which sets the Port_Binding.nat_addresses
can be simplied. This patch does this. This would help in
upcoming commits to set the nat_addresses column with the
mac and IPs of distributed logical router ports and logical
router ports with 'reside-on-redirect-chassis' set.

Acked-by: Dumitru Ceara <dceara at redhat.com>
Signed-off-by: Numan Siddique <nusiddiq at redhat.com>
Signed-off-by: Ben Pfaff <blp at ovn.org>


  Commit: ed198fb3b92e2a0b1f594c22280803bfc2f66029
      https://github.com/openvswitch/ovs/commit/ed198fb3b92e2a0b1f594c22280803bfc2f66029
  Author: Numan Siddique <nusiddiq at redhat.com>
  Date:   2019-07-03 (Wed, 03 Jul 2019)

  Changed paths:
    M ovn/northd/ovn-northd.c
    M tests/ovn.at

  Log Message:
  -----------
  ovn: Send GARP for the router ports with reside-on-redirect-chassis options set

With the commit [1], the routing for the provider logical switches
connected to a router is centralized on the master gateway chassis
(if the option - reside-on-redirect-chassis) is set. When the
failover happens and a standby gateway chassis becomes master,
it should send GARPs for the router port macs. Without this, the
physical switch doesn't learn the new location of the router port macs
immediately and this could result in traffic disruption.

This patch addresses this issue so that the ovn-controller which claims the
distributed gatweway router port sends out the GARPs.

ovn-controller sends the GARPs if the Port_Binding.nat_addresses column
is set. This patch makes use of this column, instead of adding a new column
even though the name - nat_addresses seems a bit misnomer. The documentation is
updated to highlight the usage of this column.

This patch doesn't handle sending the GARPs for the gateway router port IPs.
This will be handled in a separate patch.

[1] - 85706c34d53d ("ovn: Avoid tunneling for VLAN packets redirected to a gateway chassis")

Acked-by: Dumitru Ceara <dceara at redhat.com>
Signed-off-by: Numan Siddique <nusiddiq at redhat.com>
Signed-off-by: Ben Pfaff <blp at ovn.org>


  Commit: d65586b6fa971f03d0bf4b315986b9ea12b2ebb6
      https://github.com/openvswitch/ovs/commit/d65586b6fa971f03d0bf4b315986b9ea12b2ebb6
  Author: Numan Siddique <nusiddiq at redhat.com>
  Date:   2019-07-03 (Wed, 03 Jul 2019)

  Changed paths:
    M ovn/northd/ovn-northd.c
    M tests/ovn.at

  Log Message:
  -----------
  ovn: Send GARP for router port IPs of a router port connected to bridged logical switch

This patch handles sending GARPs for

 - router port IPs of a distributed router port

 - router port IPs of a router port which belongs to gateway router
   (with the option - redirect-chassis set in Logical_Router.options)

Acked-by: Dumitru Ceara <dceara at redhat.com>
Signed-off-by: Numan Siddique <nusiddiq at redhat.com>
Signed-off-by: Ben Pfaff <blp at ovn.org>


Compare: https://github.com/openvswitch/ovs/compare/a3173ee14768...d65586b6fa97


More information about the git mailing list