The ECMP group and member ids written into the lr_in_ip_routing and
lr_in_ip_routing_ecmp logical flows depended on the order in which
routes were processed, so a forced or fallback recompute could rewrite
those flows even though no route had changed.  Patch 1 makes the ids
depend only on the set of routes.  Patch 2 switches the remaining
nullable_string_is_equal() callers in OVN to the nullable_strcmp()
helper added by patch 1.

v2:
  - Patch 1: moved nullable_strcmp() to lib/ovn-util.h and documented
    it; clarified when ecmp_groups_node_cmp() can return 0.
  - Patch 2: new, suggested in the v1 review.

Lucas Vargas Dias (2):
  northd: Make ECMP group and members independent of processing order.
  northd: Use nullable_strcmp() instead of nullable_string_is_equal().

 lib/ovn-util.h               |  12 +++
 northd/en-group-ecmp-route.c | 168 +++++++++++++++++++++++++++++------
 northd/lb.c                  |   4 +-
 northd/lflow-mgr.c           |   2 +-
 northd/northd.c              |   3 +-
 tests/ovn-northd.at          |  66 ++++++++++++++
 6 files changed, 224 insertions(+), 31 deletions(-)

-- 
2.43.0


-- 




_'Esta mensagem é direcionada apenas para os endereços constantes no 
cabeçalho inicial. Se você não está listado nos endereços constantes no 
cabeçalho, pedimos-lhe que desconsidere completamente o conteúdo dessa 
mensagem e cuja cópia, encaminhamento e/ou execução das ações citadas estão 
imediatamente anuladas e proibidas'._


* **'Apesar do Magazine Luiza tomar 
todas as precauções razoáveis para assegurar que nenhum vírus esteja 
presente nesse e-mail, a empresa não poderá aceitar a responsabilidade por 
quaisquer perdas ou danos causados por esse e-mail ou por seus anexos'.*



_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to