mbox series

[ovs-dev,ovn,0/2] fix ARP processing for FIP traffic

Message ID cover.1590443438.git.lorenzo.bianconi@redhat.com
Headers show
Series fix ARP processing for FIP traffic | expand

Message

Lorenzo Bianconi May 25, 2020, 9:55 p.m. UTC 
Lorenzo Bianconi (2):
  Revert "Manage ARP process locally in a DVR scenario"
  northd: manage ARP request locally for FIP traffic

 northd/ovn-northd.8.xml | 49 ++++++++--------------------
 northd/ovn-northd.c     | 71 +++++++++++------------------------------
 tests/ovn.at            | 42 ++++++++++++++----------
 tests/system-ovn.at     | 28 ++++++++++++++++
 4 files changed, 85 insertions(+), 105 deletions(-)

Comments

Han Zhou May 26, 2020, 3:27 a.m. UTC | #1 
On Mon, May 25, 2020 at 2:55 PM Lorenzo Bianconi <
lorenzo.bianconi@redhat.com> wrote:
>
> Lorenzo Bianconi (2):
>   Revert "Manage ARP process locally in a DVR scenario"
>   northd: manage ARP request locally for FIP traffic
>
>  northd/ovn-northd.8.xml | 49 ++++++++--------------------
>  northd/ovn-northd.c     | 71 +++++++++++------------------------------
>  tests/ovn.at            | 42 ++++++++++++++----------
>  tests/system-ovn.at     | 28 ++++++++++++++++
>  4 files changed, 85 insertions(+), 105 deletions(-)
>
> --
> 2.26.2
>

Thanks Lorenzo for the refactor. It looks clean now and I applied the first
patch. I had some comments on patch 2. It would be helpful if @Dumitru Ceara
<dceara@redhat.com> could take a look as well since he review the earlier
version.