mbox series

[ovs-dev,v3,0/3] handle pb->chassis and pb->up from if-status module

Message ID 20231030091022.2397676-1-xsimonar@redhat.com
Headers show
Series handle pb->chassis and pb->up from if-status module | expand

Message

Xavier Simonart Oct. 30, 2023, 9:10 a.m. UTC 
Xavier Simonart (3):
  binding: slight refactor if no local binding in consider_iface_release
  binding: handle pb->chassis and pb->up from if-status module
  Fix flows not removed in ha migration

 controller/binding.c        | 221 ++++++++++++++++++++++--------------
 controller/binding.h        |  17 +++
 controller/if-status.c      | 137 +++++++++++++++++-----
 controller/if-status.h      |   9 +-
 controller/ovn-controller.c |   6 +-
 tests/ovn.at                |  94 +++++++++++++++
 6 files changed, 364 insertions(+), 120 deletions(-)

Comments

Dumitru Ceara Nov. 17, 2023, 2:25 p.m. UTC | #1 
On 10/30/23 10:10, Xavier Simonart wrote:
> Xavier Simonart (3):
>   binding: slight refactor if no local binding in consider_iface_release
>   binding: handle pb->chassis and pb->up from if-status module
>   Fix flows not removed in ha migration
> 
>  controller/binding.c        | 221 ++++++++++++++++++++++--------------
>  controller/binding.h        |  17 +++
>  controller/if-status.c      | 137 +++++++++++++++++-----
>  controller/if-status.h      |   9 +-
>  controller/ovn-controller.c |   6 +-
>  tests/ovn.at                |  94 +++++++++++++++
>  6 files changed, 364 insertions(+), 120 deletions(-)
> 

Thanks, Xavier and Mark!  It seems to me all the concerns Mark had on v2
were addressed in v3.  I applied the patch to main and backported it to
23.09.  Backporting to 23.06 generated non-trivial conflicts.  Do you
have time to prepare a backport for branches <=23.06 (if needed)?

Regards,
Dumitru